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.

Sign up for GitHub

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

Jump to bottom

[Feature Request] 通知条件設定の自動保存 #788

Open
YumNumm opened this issue Aug 18, 2024 · 0 comments
Open

[Feature Request] 通知条件設定の自動保存 #788

YumNumm opened this issue Aug 18, 2024 · 0 comments

Comments

@YumNumm
Copy link
Owner

YumNumm commented Aug 18, 2024

概要

  • 現在、通知条件を変更したあとに FloatingActionButtonの「保存する」ボタンを押下することで サーバに設定が同期される仕様になっている
  • 変更時に自動で同期されるようにしたい
    • 地域レベルの設定変更ごとにAPI通信を走らせると リクエスト数が増える上、あまりUXが良くないと思う
    • なので、ある程度まとまったタイミングで送るようにする
      • 「地震情報の通知条件設定ページ」or「緊急地震速報の通知条件設定」から「通知条件設定ページ」 に遷移した時あたりがちょうどよさそう
YumNumm added a commit that referenced this issue Aug 23, 2024
@YumNumm
Auto sync notification settings
1542d3b 
Fixes #788

Implement automatic synchronization of notification settings after changes.

* **Automatic Synchronization**:
  - Add a `ref.listen` method in `app/lib/feature/settings/features/notification_remote_settings/data/notification_remote_settings_notifier.dart` to listen for changes in the notification settings and automatically invoke the `save` method.
* **UI Changes**:
  - Remove the `FloatingActionButton` used to manually save notification settings in `app/lib/feature/settings/features/notification_remote_settings/ui/notification_remote_settings_page.dart`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/YumNumm/EQMonitor/issues/788?shareId=XXXX-XXXX-XXXX-XXXX).
@YumNumm YumNumm mentioned this issue Aug 23, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Auto sync notification settings YumNumm/EQMonitor
1 participant
@YumNumm