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: integrate show notification button option #5302

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

migcarde
Copy link

@migcarde migcarde commented May 9, 2024

Feature Preview

close #4847

Grabacion.2024-05-09.205742.mp4

PR Checklist

  • My code adheres to AppFlowy's Conventions
  • I've listed at least one issue that this PR fixes in the description above.
  • I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • All existing tests are passing.

@CLAassistant
Copy link

CLAassistant commented May 9, 2024

CLA assistant check
All committers have signed the CLA.

Copy link

github-actions bot commented May 9, 2024

🥷 Ninja i18n – 🛎️ Translations need to be updated

Project /project.inlang

lint rule new reports level link
Missing translation 50 warning contribute (via Fink 🐦)

@LucasXu0 LucasXu0 requested a review from Xazin May 10, 2024 02:10
migcarde and others added 3 commits May 10, 2024 19:45
…widgets/settings_notifications_view.dart

Co-authored-by: Mathias Mogensen <42929161+Xazin@users.noreply.github.com>
…ions/widgets/notification_button.dart

Co-authored-by: Mathias Mogensen <42929161+Xazin@users.noreply.github.com>
…otifications/notification_settings_cubit.dart

Co-authored-by: Mathias Mogensen <42929161+Xazin@users.noreply.github.com>
@Xazin
Copy link
Collaborator

Xazin commented May 13, 2024

Seems there are some issues due to merge.

You might have to rename the usage of FlowySettingListTile to SettingListTile.

@migcarde
Copy link
Author

Seems there are some issues due to merge.

You might have to rename the usage of FlowySettingListTile to SettingListTile.

Resolved naming with this Widget.

@Xazin Xazin added the v0.5.9 label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FR] Option to not show notifications icon, when there isnt any notification
3 participants