Skip to content

Currency Fixes #22

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.

Sign up for GitHub

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

Jump to bottom
Merged
brainiac merged 2 commits into from
Jan 21, 2026
Merged

Currency Fixes #22

brainiac merged 2 commits into from
Jan 21, 2026

Conversation

@Sicprofundus
Copy link
Contributor

@Sicprofundus Sicprofundus commented Jan 20, 2026

  • added missing "LOGOFVENTURES"
  • updated enum entries to match dbstr_texts where possible

@Sicprofundus
Currency Fixes
350618d 
- added missing "LOGOFVENTURES"
- updated enum entries to match dbstr_texts where possible
brainiac
brainiac requested changes Jan 21, 2026
View reviewed changes
Copy link
Contributor

@brainiac brainiac left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Renaming these makes this unmergable, as then I would have to go and update all the branches including emu to match it.

Can you just stick with the new value?

@Sicprofundus
Copy link
Contributor Author

Sicprofundus commented Jan 21, 2026

sounds good

@Sicprofundus Sicprofundus requested a review from brainiac January 21, 2026 17:34
@brainiac brainiac merged commit 3a09f07 into macroquest:live Jan 21, 2026
brainiac pushed a commit that referenced this pull request Jan 21, 2026
@Sicprofundus @brainiac
Currency Fixes (#22)
38f966e 
* Currency Fixes

- added missing "LOGOFVENTURES"
- updated enum entries to match dbstr_texts where possible

* revert the previous naming update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@brainiac brainiac Awaiting requested review from brainiac

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Sicprofundus @brainiac