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

Open notifications view of the correct account #2003

Closed
juliusknorr opened this issue Jan 19, 2018 · 8 comments
Closed

Open notifications view of the correct account #2003

juliusknorr opened this issue Jan 19, 2018 · 8 comments
Labels
enhancement feature: activity and notification Server activity and notifications

Comments

@juliusknorr
Copy link
Member

juliusknorr commented Jan 19, 2018

Actual behaviour

  • No notifications visible since the wrong account A is active

Expected behaviour

  • The correct list of notifications of account B shoud be visible

Steps to reproduce

  1. Have two accounts setup
  2. Have account A open
  3. Receive a notification on account B
  4. Click the notification popping up
@tobiasKaminsky
Copy link
Member

What kind of notifications are you talking about?
Push notifications? Upload/download notifications?

@juliusknorr
Copy link
Member Author

Push notifications. 😉

@tobiasKaminsky
Copy link
Member

@mario is this handled somewhere else?

private void sendNotification(String contentTitle) {

here I do not find any reference to account?

@mario
Copy link
Contributor

mario commented Jan 22, 2018

It's not. It'll be once we do the decryption.

@tobiasKaminsky
Copy link
Member

But then notification should be shown on all accounts, or?

@mario
Copy link
Contributor

mario commented Jan 22, 2018

Once you open the notifications screen, it'll only fetch notifications for your current account. Which is not that great UX-wise, but like I said ... this will be fixed in 3.1 :)

@tobiasKaminsky
Copy link
Member

Ah. Sorry, I read it wrong. I read that only push notifications from account A are received if A as active and not received if B is active.

@joshtrichards joshtrichards added the feature: activity and notification Server activity and notifications label Oct 9, 2023
@joshtrichards joshtrichards changed the title Open notifications view of the correct account Open push notifications view of the correct account Dec 8, 2023
@joshtrichards joshtrichards changed the title Open push notifications view of the correct account Open notifications view of the correct account Dec 8, 2023
@joshtrichards
Copy link
Member

I guess this was apparently not addressed in 3.1 or maybe there was a regression somewhere along the way.

Consolidating tracking into #10495.

@joshtrichards joshtrichards closed this as not planned Won't fix, can't repro, duplicate, stale Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: activity and notification Server activity and notifications
Projects
None yet
Development

No branches or pull requests

4 participants