Skip to content

Implemented history page night selection for deletion#62

Merged
MouradLachhab merged 9 commits intomasterfrom
history
Nov 12, 2020
Merged

Implemented history page night selection for deletion#62
MouradLachhab merged 9 commits intomasterfrom
history

Conversation

@MouradLachhab
Copy link
Copy Markdown
Contributor

It is now possible to select night for deletion.

image

image

Comment thread mobile/lib/src/application/sleep_history/sleep_history_cubit.dart Outdated
Comment thread mobile/lib/src/domain/sleep_history/i_sleep_history_repository.dart Outdated
Comment thread mobile/lib/src/domain/sleep_history/i_sleep_history_repository.dart Outdated
Comment thread mobile/lib/src/application/sleep_history/sleep_history_cubit.dart Outdated
Comment thread mobile/lib/src/infrastructure/sleep_history/sleep_history_repository.dart Outdated
Comment thread mobile/lib/src/presentation/pages/sleep_history_page/sleep_history_page.dart Outdated
Comment thread mobile/lib/src/application/sleep_history/sleep_history_cubit.dart Outdated
Comment thread mobile/lib/src/application/sleep_history/sleep_history_cubit.dart Outdated
Comment thread mobile/lib/src/infrastructure/sleep_history/sleep_history_repository.dart Outdated
Comment thread mobile/lib/src/presentation/pages/sleep_history_page/sleep_history_list.dart Outdated
Comment thread mobile/lib/src/presentation/pages/sleep_history_page/sleep_history_page.dart Outdated
Comment thread mobile/lib/src/presentation/pages/sleep_history_page/sleep_history_page.dart Outdated
Copy link
Copy Markdown
Contributor

@abelfodil abelfodil left a comment

Choose a reason for hiding this comment

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

Peux-tu lancer un format pls?

@MouradLachhab
Copy link
Copy Markdown
Contributor Author

Peux-tu lancer un format pls?

Running flutter format **/*.dart did no changes except add one newline on one file. What changes were you looking for ? Maybe there are some linting settings that I don't have set.

@mateobelanger
Copy link
Copy Markdown
Contributor

Quand on visionne une nuit en particulier faudrait que le bouton du drawer devienne un bouton (flèche) pour retourner à la liste des nuits. C'est pas super important, mais niveau ergonomie de devoir rouvrir le drawer et retourner dans History c'est pas top.

@override
void initializeRepository() {
SleepHistoryRepository() {
_sleepHistoryPersistency.add(mock_data_1);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Si le Repository est mock je propose qu'on le renomme MockSleepHistoryRepository. Lorsqu'on voudra implémenter la vrai version, on aura juste à créer une nouvelle classe SleepHistoryRepository et l'injecter dans l'app à la place de MockSleepRepository.

}

emit(SleepSequenceHistoryEditInProgress(
_sleepHistoryRepository.getSleepSequences(), _selectedSequences));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

J'imagine que ce sera async. Il faudrait probablement utiliser Future ISleepHistoryRepository::getSleepSequences(); J'imagine que les ajouts et suppression de la persistence retourneront aussi un Future.

Comment thread mobile/lib/src/infrastructure/sleep_history/sleep_history_repository.dart Outdated
@MouradLachhab MouradLachhab merged commit bf5d577 into master Nov 12, 2020
@MouradLachhab MouradLachhab deleted the history branch November 12, 2020 17:02
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.

4 participants