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

inbox 接收到新訊息時 需要有數字通知 #190

Closed
scott-dt opened this issue Nov 13, 2020 · 5 comments
Closed

inbox 接收到新訊息時 需要有數字通知 #190

scott-dt opened this issue Nov 13, 2020 · 5 comments
Assignees
Labels
uiux UI or UX update workaround Fixed with a workaround
Milestone

Comments

@scott-dt
Copy link

scott-dt commented Nov 13, 2020

capture lite inbox 顯示目前有多少待收 postcapture

@scott-dt scott-dt added this to the backlog milestone Nov 13, 2020
@seanwu1105 seanwu1105 added the uiux UI or UX update label Nov 17, 2020
@scott-dt scott-dt modified the milestones: backlog, beta7 Nov 18, 2020
@scott-dt scott-dt modified the milestones: beta7, beta7.1, backlog Nov 19, 2020
@seanwu1105
Copy link
Contributor

WORKAROUND: polling to get badge count.

I implemented this feature by polling because:

  1. Users might ignore multiple transactions in app, and thus the inbox_count field does not help. App still needs to request the inbox list.
  2. If the notification got pushed while the user is not using the app, the badge will never be updated as the update is outside of ngZone.

Ultimately, we need to apply repository pattern (see #226) to fix these problem, but I do not have additional time before the release.

See 6c5032c.

@seanwu1105 seanwu1105 added fix/feature committed workaround Fixed with a workaround labels Nov 30, 2020
@scott-dt
Copy link
Author

scott-dt commented Dec 3, 2020

201202 demo 會議提出將數字通知改為紅點通知

@scott-dt
Copy link
Author

scott-dt commented Dec 4, 2020

@seanwu1105 目前還是數字通知 須改為紅點顯示不要數字 ( 功能正常有正常顯示 )

@scott-dt
Copy link
Author

scott-dt commented Dec 9, 2020

beta7.3 進行測試

@scott-dt scott-dt modified the milestones: beta7.1, beta7.3 Dec 9, 2020
@scott-dt
Copy link
Author

0.11.7 Android 於接收後會正常顯示 ios 仍有 issues 需要將 app 關閉在重啟才會顯示 #365

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
uiux UI or UX update workaround Fixed with a workaround
Projects
None yet
Development

No branches or pull requests

3 participants