Fixed recipe names, enhanced a recipe icon, updated the readme #1
No reviewers
Labels
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: cacklingfiend/IntermediatesForYou2#1
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "Hraukr/IntermediatesForYou2:main"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fixed the recipe names for bronze-spring and elementium-heat-shielding.
Enhanced the bronze-spring icon.
Updated the readme.md file for the mod (Suitable for use on the mod page description as well)
Tested in-game with my current save and the changes seem to have taken.
This is my first pull request ever! I wanted to learn a bit of how Factorio works and wanted to see if I could help out in a way. Let me know if you have any comments or if I should do anything different.
Thanks for the PR. On small suggestion but I'll happily accept without any further change.
@ -203,3 +203,3 @@localised_name = { "item-name.bronze-spring" },category = "crafting",order = "s2[spring]",icons = (mods["bztin2"] andYou could use
data.raw.item["bronze-plate"].icon or data.raw.item["bronze-plate"].icons and data.raw.item["bronze-plate"].icons[1](maybe make the item a variable) to be a bit more generic.The bronze-spring recipe has been updated to use any existing bronze-spring item icon. Plus a few other changes.
Thanks for the input. I looked at some of the other mods that had the overlaid icons to see how they did it and copied the syntax. Appreciate the suggestion with a better method and will look to integrate it shortly.
Feel free to accept the PR now if you wish, or wait until it has been updated with your suggestion.
View command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.