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

feat: extensible subscriptions #63

Merged
merged 14 commits into from Sep 28, 2023
Merged

feat: extensible subscriptions #63

merged 14 commits into from Sep 28, 2023

Conversation

imorland
Copy link
Member

@imorland imorland commented Aug 23, 2023

Introduces a new extensible SubscriptionModal to select the tag subscription options, in place of the old menu.

Added a new event SubscriptionChanging, which developers can listen for in order to extend the subscription behaviour

Removes SubscriptionMenuItem and SubscriptionMenu

image

@imorland imorland changed the title Im/notifications feat: Allow other extensions to interact with the notifications before they are sent from the queue Aug 23, 2023
@imorland imorland changed the title feat: Allow other extensions to interact with the notifications before they are sent from the queue feat: extensible subscriptions Sep 12, 2023
@imorland imorland merged commit 4428fd2 into master Sep 28, 2023
4 checks passed
@imorland imorland deleted the im/notifications branch September 28, 2023 11:14
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.

None yet

2 participants