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

🦑 [App Badges] Feature #697

Merged
merged 4 commits into from
Jun 17, 2019
Merged

Conversation

justinswart
Copy link
Contributor

@justinswart justinswart commented May 31, 2019

πŸ“² What

Brings improved app badging to the app.

πŸ”€ Merged so far:

#694 Display count on activities tab bar item

0 unread 5 unread 99 unread > 99 unread
Screen Shot 2019-05-30 at 2 38 12 PM Screen Shot 2019-05-30 at 2 36 59 PM Screen Shot 2019-05-30 at 2 37 28 PM Screen Shot 2019-05-30 at 2 37 50 PM

#698 Update badge value on push and lifecycle events

Push in app, clear on tap Count from user refresh Clear on activities refresh Clear on logout
no-push-count-from-user-clear-on-tap clear-on-refresh clear-on-logout

#706 Improve voice-over support for badge value

⏰ TODO

  • Clear count on back-end when tapping on activities
  • a11y for badge count voice over
  • Updating count when a push notification is received while the app is open

* Set badge value on app launch based on current applicationIconBadgeNumber

* Add MockApplication.swift

* Move constant to within function, improve coalescing

* SwiftFormat
* Update and clear unseen activity count on lifecycle events, push, user session

* Update badge value on current user updated instead of session started, clear value on activities refresh

* Fix tests

* Only clear activity count when we know that one is set, set badge value while on activities tab, update user in environment

* Remove redundant withEnvironment

* Prevent infinite loop

* Camel-case test functions

* Do not update user in environment if logged out

* Add logged out test
@ksr-ci-bot
Copy link

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

* Add Strings.activities_badge_value_plus

* Handle badge value correctly when voice over is enabled
@justinswart justinswart marked this pull request as ready for review June 17, 2019 20:13
@justinswart justinswart merged commit a76f86a into master Jun 17, 2019
@justinswart justinswart deleted the feature-activity-badge-icon branch June 17, 2019 21:18
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.

None yet

3 participants