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

Nanochat Improvements and Lookup (DV Port) #1692

Merged

Conversation

honeyed-lemons
Copy link

Ports two Nanochat PRs from DeltaV

Changelog
🆑 Toby222, Honeyed_Lemons_

  • add: Adds Nanochat Lookup! Look up your friends in your Nanochat app today!
  • tweak: The Nanochat ui has been spruced up a lot, look at keybinds too!

Toby222 and others added 12 commits February 14, 2025 19:14
* NanoChat Phonebook MVP

* Only work with active Telecomms server with correct channel

* Add localization and program sprite

* Cartridge icon

* Only preinstall on HoP, Courier, and QM PDAs

* untroll, fix formatting

* fix double newline

* replace NanoChat lookup cartridge with UI elements

* List Number setting in NanoChat instead of PDA

* replace NanoChat lookup cartridge with UI elements

* re-fuck formatting just to make the diff on GitHub nicer

* remove superfluous svg

* remove leftovers I somehow created

* address simple requested changes; remove more leftovers

* NanoChat lookup to own Container

* fix indentation

(cherry picked from commit 27383be3dc9a66c409d444b59601c344b013576a)
@honeyed-lemons honeyed-lemons changed the title Nanochat Improvements (DV Port) Nanochat Improvements and Lookup (DV Port) Feb 15, 2025
Copy link

@kipdotnet kipdotnet left a comment

Choose a reason for hiding this comment

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

functions perfectly as far as I can tell. no errors, and fixes a bunch of annoyances. approving but wont be merged pending proper code review

@formlessnameless formlessnameless merged commit 25d9e04 into impstation:master Feb 17, 2025
4 checks 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.

5 participants