Skip to content

feat: Main street shop unlocking#106

Open
Le0X8 wants to merge 9 commits intomarcrobledo:masterfrom
Le0X8:master
Open

feat: Main street shop unlocking#106
Le0X8 wants to merge 9 commits intomarcrobledo:masterfrom
Le0X8:master

Conversation

@Le0X8
Copy link

@Le0X8 Le0X8 commented Jan 28, 2026

I've found out how the shops on the main street are unlocked, so I added this feature to the "Shops" tab :)

image

Everything is tested on ACNL:WA, but the old version should work as well.

@marcrobledo
Copy link
Owner

Wow! This looks so cool, thank you!

I'll be merging it for sure. But before that, I want to be sure the review I just posted is correct.

@Le0X8
Copy link
Author

Le0X8 commented Feb 2, 2026

@marcrobledo

I want to be sure the review I just posted is correct.

I'm sorry, but what review are you talking about? I hope this question is not as stupid as it sounds.

image

@marcrobledo
Copy link
Owner

@marcrobledo

I want to be sure the review I just posted is correct.

I'm sorry, but what review are you talking about? I hope this question is not as stupid as it sounds.
image

This :-)

@Le0X8
Copy link
Author

Le0X8 commented Feb 16, 2026

am I missing something? There's nothing here...

this.shopHarvey.save();

savegame.writeU8(Offsets.UNLOCK_NOOK, this.shopNookUnlock);
savegame.writeU8(Offsets.UNLOCK_NOOK+1, this.shopNookUnlock);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this intentional?
Are we sure the following byte to UNLOCK_NOOK is always a duplicate of the previous one?

I'd rather prefer to make this second byte so it only gets updated if user has changed the value in the select dropdown.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As far as I was able to tell, this is used for expansions if you have spent enough bells, therefore it would totally make sense only to overwrite this if the user wants to change the building.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me know when you've done the edit so I can merge it :-)

@marcrobledo
Copy link
Owner

am I missing something? There's nothing here...

Sorry, looks like I missed clicking on the Start review. It was odd because I could see the comment being there and even a reply button.

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