Skip to content

NotificationsList: remove app entries hashmap - #332

Merged
danirabbit merged 9 commits into
mainfrom
danirabbit/notificationslist-rmappentries
Sep 3, 2026
Merged

danirabbit merged 9 commits into
mainfrom
danirabbit/notificationslist-rmappentries

Conversation

@danirabbit

Copy link
Copy Markdown
Member

No description provided.

@danirabbit
danirabbit marked this pull request as ready for review September 3, 2026 21:11
@danirabbit
danirabbit requested a review from a team September 3, 2026 21:11
@danirabbit danirabbit added this to OS 9 Sep 3, 2026
@zeebok

zeebok commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Is this supposed actually remove the hashmap member from NotificationsList or this setup to do that in a future PR?

@danirabbit

Copy link
Copy Markdown
Member Author

@zeebok oops good catch!

@danirabbit
danirabbit merged commit b255d6c into main Sep 3, 2026
2 of 4 checks passed
@github-project-automation github-project-automation Bot moved this to Done in OS 9 Sep 3, 2026
@danirabbit
danirabbit deleted the danirabbit/notificationslist-rmappentries branch September 3, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants