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

Notifications are not shown when activity app is disabled in the server. #4651

Closed
SergioBertolinSG opened this issue Apr 8, 2016 · 3 comments
Assignees
Labels
ReadyToTest QA, please validate the fix/enhancement sev2-high type:bug
Milestone

Comments

@SergioBertolinSG
Copy link

Steps to reproduce

Having connected a server to the client do:

  1. Disable activity app in the server.
  2. Enable announcements app in the server.
  3. Run cron.php in the server.
  4. Sync desktop client using admin user.
  5. Create a notification with another user "admin2" included in admin group.
  6. Open desktop client. Look for notifications.

Expected behaviour

You find somewhere the notifications.

Observed behaviour

Notifications are not there.

(They would be in activity tab if it were enabled in the server).

Client Version
Version 2.2.0-nightly20160408 (build 3250).

Operating system
OS X El capitán 10.11.
Windows 10.

@SergioBertolinSG SergioBertolinSG added this to the 2.2.0-current milestone Apr 8, 2016
@SergioBertolinSG SergioBertolinSG changed the title Notifications are not shown where activity app is disabled in the server. Notifications are not shown when activity app is disabled in the server. Apr 8, 2016
@nickvergessen
Copy link
Contributor

Side note, many bigger installations have activities disabled due to the performance slow done it causes for actions on shared files

dragotin pushed a commit that referenced this issue Apr 11, 2016
@dragotin dragotin added the ReadyToTest QA, please validate the fix/enhancement label Apr 11, 2016
@dragotin dragotin assigned draht and dragotin and unassigned draht Apr 11, 2016
dragotin pushed a commit that referenced this issue Apr 13, 2016
@ckamm
Copy link
Contributor

ckamm commented Apr 14, 2016

Testing:

  • The Server Activity tab did appear when a notification was received
  • When I have a single account to a server that has activity disabled, the Server Activity tab is still shown. It goes away when I click it.

I think the problem is the unconditional emit hideActivityTab(false); even if _widgetForNotifId.isEmpty(). But we also don't want to accidentally hide it if activities are enabled. That needs some care.

@ckamm ckamm closed this as completed in dfd7d4d Apr 14, 2016
@ckamm
Copy link
Contributor

ckamm commented Apr 14, 2016

Closing this through the commit was unintentional, but I am done testing this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ReadyToTest QA, please validate the fix/enhancement sev2-high type:bug
Projects
None yet
Development

No branches or pull requests

5 participants