search

zulip / zulip-flutter

Upcoming Zulip mobile apps for Android and iOS, using Flutter
Apache License 2.0
200 stars 193 forks source link

notif: Support migration of Android notification channels #981

Closed rajveermalviya closed 1 month ago

rajveermalviya commented 1 month ago

(Split from #717)

Needed for #340, when updating notification channels to use a custom notification sound.

rajveermalviya commented 1 month ago

Thanks for the review @chrisbobbe! Pushed a new revision PTAL.

chrisbobbe commented 1 month ago

Thanks! LGTM, over to @gnprice.

rajveermalviya commented 1 month ago

Thanks for the reviews @chrisbobbe and @gnprice! @gnprice pushed a new revision with update test bindings. PTAL.

gnprice commented 1 month ago

Thanks for the revision! All looks good — merging.