Skip to content

Conversation

SicklySilverMoon
Copy link
Contributor

This will be used in a board I've just finished so I'd like to be here as well, thanks!

@Sophilautia
Copy link
Contributor

Are you uploading the board as well?

@SicklySilverMoon
Copy link
Contributor Author

Thought that had to be done separately for some reason, I'll get working on that quick

@SicklySilverMoon
Copy link
Contributor Author

Added!

@Sophilautia
Copy link
Contributor

Yaml had some formatting issues that I fixed, namely removing superfluous contents, fixing up the description formatting, and removing the --- from the start of the yaml.

Looking at the pathing, I'm pretty sure it's all formatted correctly? This design is very.. Ideosyncratic! I'm not sure what the logic of the doors are. Usually you'd have the colors match between the two connecting doors. But it seems like it all works fine. Can I ask that you change the colors of the doors perhaps?

The yaml also lacks an author and changelog section, so if you could add those that would be appreciated!

@SicklySilverMoon
Copy link
Contributor Author

I can change one of the greens to a yellow, but I would greatly prefer to keep the red and blue door mismatch as it mimics a notable part of the game imo:
image
Although if that doesn't work I can swap it out for 2 sets of doors, one blue and one red in a similar configuration

@SicklySilverMoon
Copy link
Contributor Author

How's this?
image
I think this is actually preferred since I discovered an issue with the one way doors that players couldn't go up into the red one on the old map under certain conditions (weird tile placement I guess). They're mapped in a one way path style that prevents an infinite loop so there shouldn't be an issue (I'll rerun all the tests if this is all good).

Author info, some district name changes
@Sophilautia
Copy link
Contributor

Yeah that looks good to me

@SicklySilverMoon
Copy link
Contributor Author

Uploaded the new board and image after some testing, everything appears to work correctly as expected since its functionally identical. I believe this should be it?

@Sophilautia Sophilautia merged commit 75206c8 into FortuneStreetModding:main Mar 13, 2025
1 check passed
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.

2 participants