Skip to content

[v2.0] Add toast notifications#312

Open
jaylikesbunda wants to merge 4 commits into
two-point-zerofrom
ui-toast
Open

[v2.0] Add toast notifications#312
jaylikesbunda wants to merge 4 commits into
two-point-zerofrom
ui-toast

Conversation

@jaylikesbunda
Copy link
Copy Markdown
Member

What's new (Author - fill this out)

Changed

  • Added toast notification system

Removed

Verification (Author - fill this out)

  • I have tested Primary device(s) (list exact HW + config): T-Deck Plus
  • I have tested potentially affected devices (and/or list potential affected devices not available to you):
  • I have wrapped device specific code with #ifdef CONFIG_... or similar: [ ] Yes / [ ] N/A
  • I have updated Hugo Docs with any new or changed info for end users: [ ] Yes / [ ] N/A

Linked Issues

  • Closes #

Checklist (Reviewer - don't fill this out)

  • Code builds, flashes, and feature verified with no functional issues on listed device(s)
  • Changes reviewed for unintended impact on other devices/targets

@jaylikesbunda jaylikesbunda self-assigned this May 4, 2026
@jaylikesbunda
Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	main/core/glog.c
#	main/vendor/GPS/gps_logger.c

Co-authored-by: jaylikesbunda <133104319+jaylikesbunda@users.noreply.github.com>
Copy link
Copy Markdown

Copilot AI commented May 13, 2026

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in commit 3a34d6e8. There were two conflicting files:

  • main/core/glog.c: Kept our heap-based approach for deferred log storage (matching the existing malloc/free pattern).
  • main/vendor/GPS/gps_logger.c: Kept toast_show("GPS log saved", TOAST_SUCCESS) inside the file-close block; the base branch had moved wd_free_dedupe_tables() and glog() outside the block (making them unconditional), so those duplicates from our branch were removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants