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

Feature/#1-ringing_alarm #2

Merged
merged 2 commits into from
Jun 10, 2022
Merged

Feature/#1-ringing_alarm #2

merged 2 commits into from
Jun 10, 2022

Conversation

MrMocchy
Copy link
Collaborator

アラームの基本機能である、指定時刻に通知を送りアラームを鳴らす機能を作成。
実機とエミュレーターで(不安定ながらも)動作確認済み。
実機にインストールするための設定と、アラームの音源のフリー素材を含む。

MrMocchy added 2 commits June 10, 2022 09:55
アラームの基本機能である、指定時刻に通知を送りアラームを鳴らす機能を作成。
view と model の2ファイルに分けて作成し、その画面に遷移するためのボタンをmainに作成。
パッケージはそれぞれ>flutter pub add ~~~で入れる必要があった。
.xmlではandroidのいくつかの機能の権限をもらい、
.gradleでは実機にインストールするための設定を記述。
Copy link
Collaborator

@herring101 herring101 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
めっちゃいい感じですね!!コメントも充実していてわかりやすかったです

気になる点は

  • スリープ状態になった状態でアラームが鳴ると、タイマーが動いたまま23:59:59...となる
    スリープ状態関係で他にも不具合がありそう
  • アプリ表示時にも音を鳴らしたい
    タイマー機能はアプリ開きながら使うこともあることを考えると必要かなと

@herring101
Copy link
Collaborator

mergeしちゃって大丈夫です

@MrMocchy MrMocchy merged commit f69f16f into main Jun 10, 2022
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.

2 participants