Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve equipment grid mod compatibility #2

Merged
merged 3 commits into from
Nov 2, 2020

Conversation

ryepup
Copy link
Contributor

@ryepup ryepup commented Nov 1, 2020

Applied suggestion from Glendening to add more categories to the equipment grid and allow more modded vehicle parts to work inside spidertron.

I tested with Krastorio2.

Applied [suggestion from Glendening](https://mods.factorio.com/mod/SpidertronEngineer/discussion/5f81aaa86c91db28401e0848) to add more categories to the equipment grid and allow more modded vehicle parts to work inside spidertron.

I tested with Krastorio2.
Allow robot gear in spidertron
@tburrows13
Copy link
Owner

tburrows13 commented Nov 2, 2020

Thanks for this. Let me know if you're done and I'll merge and release. Can you confirm that it still works with non-vehicle armour? I see that you've removed the "armor", "spidertron", "vehicle", "armoured-vehicle" categories, but I don't remember much about it from when I added that bit.

@ryepup
Copy link
Contributor Author

ryepup commented Nov 2, 2020

I didn't check that, lemme revert everything I didn't directly test

@ryepup
Copy link
Contributor Author

ryepup commented Nov 2, 2020

OK, updated with only the changes I tested, without some of the other code from the mod forum.

@tburrows13 tburrows13 merged commit 081f307 into tburrows13:master Nov 2, 2020
@tburrows13
Copy link
Owner

Nice, thanks for your help!

@ryepup ryepup deleted the mod-grid-compat branch November 2, 2020 16:09
@ryepup
Copy link
Contributor Author

ryepup commented Nov 2, 2020

Thanks for the great mod!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants