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

[Activity View] If you disable the Activity app you should not see the server activity tin the activity tab #4533

Closed
cdamken opened this issue Mar 2, 2016 · 6 comments · Fixed by #4552
Assignees
Labels
blue-ticket ReadyToTest QA, please validate the fix/enhancement
Milestone

Comments

@cdamken
Copy link
Contributor

cdamken commented Mar 2, 2016

Steps to reproduce

  1. Open your ownCloud desktop client 2. Disable Activity app in the server.
  2. Go to the Desktop client
  3. Go to Open Folder "ownCloud"
  4. Create a new folder, called aaa
  5. Right click on the folder aaa
  6. Select share with ownCloud
  7. Share with other user (e.g. user1)
  8. Wait to sync

Expected behavior

The activity tab should not be shown (hidden). (If multi-account if all the servers are disabled)

Actual behavior

No message appears "The administrator has disabled the activity for this server"
Related #3732 #4163

@MorrisJobke @dragotin
00004066

@guruz guruz added this to the 2.2.0-current milestone Mar 2, 2016
dragotin pushed a commit that referenced this issue Mar 8, 2016
If the ownCloud server does not have the activity app enabled,
it returns 999 as status code. If all the configured accounts
do that, this code hides the entire tab with the server
activities.

This is supposed to fix #4533
@dragotin
Copy link
Contributor

dragotin commented Mar 9, 2016

This needs testing!

@dragotin dragotin reopened this Mar 9, 2016
@dragotin dragotin added the ReadyToTest QA, please validate the fix/enhancement label Mar 9, 2016
@SergioBertolinSG
Copy link

Not sure what is expected here.

This message "The administrator has disabled the activity for this server" doesn't appear.

But activity tab dissapears and appears correctly.

In OS X I cannot test it (at least following the exact steps), it is blocked by #4650

@guruz guruz removed the ReadyToTest QA, please validate the fix/enhancement label Apr 11, 2016
@dragotin
Copy link
Contributor

I think it is fine that there is just no tab if the activity tab is disabled. The message will appear if there is a notification and thus the Activity Tab appears.

@dragotin dragotin added the ReadyToTest QA, please validate the fix/enhancement label Apr 15, 2016
@mcastroSG
Copy link

mcastroSG commented Apr 20, 2016

Hi hi:

Steps executed

  1. As admin, disable Activity app in the server.
  2. Wait for sync.
  3. Open your ownCloud desktop client to check that Server Activity tab disappears.
  4. As admin, generate a new notification with Notification App.
  5. Wait for sync.

Results

Server Activity tab is not shown at Settings > Activity until server generates a new notification and client sync. At that moment Server Activity tab appears showing notification and the message "Accout XXXX@XXXXX:XXXX does not have activities enabled" appears at the bottom. After closing the notification and wait for next sync Server Activity tab disappears again.

MacOS
Client Version: Version 2.2.0-nightly20160420 (build 3284)
Client OS: MacOS X El Capitan 10.11.4
Server Version: 9.1 Alpha
Server OS: Ubuntu

Windows
Client Version: Version 2.2.0-nightly20160420 (build 6003)
Client OS: Windows 10
Server Version: 9.1 Alpha
Server OS: Ubuntu

Regards

@mcastroSG
Copy link

@cdamken is the current behaviour ok?

Results

Server Activity tab is not shown at Settings > Activity until server generates a new notification and client sync. At that moment Server Activity tab appears showing notification and the message "Accout XXXX@XXXXX:XXXX does not have activities enabled" appears at the bottom. After closing the notification and wait for next sync Server Activity tab disappears again.

@cdamken
Copy link
Contributor Author

cdamken commented Apr 22, 2016

@mcastroSG Looks good for me 👍 Thanks

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

Successfully merging a pull request may close this issue.

5 participants