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

not working with group folders #264

Closed
andreasmalta opened this issue Apr 18, 2018 · 26 comments · Fixed by #190
Closed

not working with group folders #264

andreasmalta opened this issue Apr 18, 2018 · 26 comments · Fixed by #190
Labels

Comments

@andreasmalta
Copy link

Steps to reproduce

  1. enable "Group folders" app
  2. create a group folder and set permissions to an existing group
  3. create a files/folders, edit files etc.

Expected behaviour
I would like the users to be notified in the activity app or by email, like it works for shared folders

Actual behaviour
in the activity stream there are no records...

Server configuration
Operating system: Centos7

Web server: Apache

Database: MariaDB

PHP version: 7.1 (ius)

Nextcloud version: 13.0.1

Where did you install Nextcloud from: https://download.nextcloud.com

Signing status: No errors have been found.

List of activated apps:
Activity
Auditing / Logging
Brute-force settings
Collabora Online
Collaborative tags
Deleted files
Draw.io
External sites
File access control
File sharing
Group folders
Impersonate
Log Reader
Monitoring
Notifications
PDF viewer
Talk
Theming
Update notification
Versions
Video player
Zimbra Drive

@ghost
Copy link

ghost commented Jun 19, 2018

Using nextcloud 13.0.4 (activity 2.6.1, group folders 1.2.0), other group members still do not receive any mail on changes inside the group folder. If you change something, you will receive a mail for your changes.

Would be nice, when everyone of the group gets informed, when there are changes within the group folder.

Thank You!

@ghost
Copy link

ghost commented Jul 11, 2018

Any news about this issue? We really need this feature working - is this issue handled in the activity-part or in the group-folders-part (nextcloud/groupfolders#112)? I would like to help getting this working soon.

@jacestk
Copy link

jacestk commented Aug 2, 2018

Yep This is a really missing function. I have created severals groups folders and only the user who has uploaded the file is receiving the notification... that's nasty.

@RuudschMaHinda
Copy link
Member

Was anyone able to take a look at this yet? This is a make or break feature for the group-folders app in one of my installations.

@noresistence
Copy link

I can confirm the issue for 15.0.5
Possibly related to #190 which does not seem to be worked on in recent past

@schmidtadrian
Copy link

this Feature would be awesome!

@tim-bo
Copy link

tim-bo commented Mar 8, 2019

I can confirm the issue for 15.0.5
Possibly related to #190 which does not seem to be worked on in recent past

i also can confirm the problem. NC15.0.5 with groupfolders 2.0.3

@joe-speedboat
Copy link

I can confirm the problem. NC15.0.5 with groupfolders 2.0.4
Sorry guys, but this is a show stopper. you work in teams shared by group folders and the activity is not visible to the members of group folders.
I cant belive that this issue is open for about a year!?
How do you handle this with your team members out there???
Is there a workaround???
Thanks Chris

@vpert
Copy link

vpert commented Mar 28, 2019

tis can't be a "feature" this is a basic requirement when running a shared storage platform. Really bad no one is taking care on this.
Tom

@Fin3
Copy link

Fin3 commented Apr 6, 2019

We need this feature too!

@Mululu
Copy link

Mululu commented Jul 16, 2019

Push!

We're is the Support Team?

@sscholl
Copy link

sscholl commented Aug 17, 2019

nobody cares?

@nickvergessen
Copy link
Member

We do care, we are just out of ideas:
#190

@izzetaydin83
Copy link

Any progress about this issue ?

@izzetaydin83
Copy link

Find a way to resolve issue:

mysql -u root -D -e "select from_unixtime(timestamp),type,user,SUBSTR(SUBSTRING_INDEX(oc_ldap_user_mapping.ldap_dn,',',1),4) ,app,file,object_type from oc_activity LEFT join oc_ldap_user_mapping ON (oc_activity.user=oc_ldap_user_mapping.owncloud_name) where timestamp > $epoch"

where epoch is epochtime-86400 gives me result of daily activity

@hriekehof
Copy link

hriekehof commented Jan 2, 2020

hi. If the fix @izzetaydin83 does work is there a chance that this can be implemented ?
A fix for this would be really awesome since an activity feed is something everyone expects nowadays from a file sharing platform :)

@Andreas-Kainz
Copy link

If the issue is fixed, close it.

@kokotko1337
Copy link

@DarkknightAK it is not fixed

@sammo2828
Copy link

sammo2828 commented Feb 3, 2020

Find a way to resolve issue:

mysql -u root -D -e "select from_unixtime(timestamp),type,user,SUBSTR(SUBSTRING_INDEX(oc_ldap_user_mapping.ldap_dn,',',1),4) ,app,file,object_type from oc_activity LEFT join oc_ldap_user_mapping ON (oc_activity.user=oc_ldap_user_mapping.owncloud_name) where timestamp > $epoch"

where epoch is epochtime-86400 gives me result of daily activity

@izzetaydin83

This doesn't resolve the issue so please don't write as such. Statements like this misleads most people trying out out nextcloud!

It's a workaround for particular users with mysql root access, which is laughable at best 😃

@phonon112358
Copy link

@nickvergessen @rullzer When will #190 be reviewed? and when will it backported to NC 18? Any ETA???

@phonon112358
Copy link

phonon112358 commented May 5, 2020

In the meantime, I have manually applied the changes proposed in #190 to my NS 18 instance ... It works like a charm!!! @nickvergessen @wiswedel

@Mululu
Copy link

Mululu commented May 5, 2020

@phonon112358

In the meantime, I have manually applied the changes proposed in #190 to my NS 18 instance ... It works like a charm!!! @nickvergessen @wiswedel

Can you briefly explain what you have to do to make this work?

@phonon112358
Copy link

You only need to change the /var/www/nextcloud/apps/activity/lib/FilesHooks.php according to https://github.com/nextcloud/activity/pull/190/files#diff-dbc5f6ba450710b214d9d73b15c9f186 (I just copied the changed file from the pull request and overwrote the existing file).
The second file from the PR, you don't even need! ;)

It was really simple...

@diemelone
Copy link

diemelone commented Apr 27, 2021

You only need to change the /var/www/nextcloud/apps/activity/lib/FilesHooks.php according to https://github.com/nextcloud/activity/pull/190/files#diff-dbc5f6ba450710b214d9d73b15c9f186 (I just copied the changed file from the pull request and overwrote the existing file).
The second file from the PR, you don't even need! ;)

It was really simple...

@nickvergessen is that still the official fix? Because it does not work for me...

I have tried the other fix from you I found here #190 (comment)
Groupfolders now show up in the activity app but mails are not sent
For Activity 1.10.0 the fix was applied to line 234 in lib/FilesHooks.php
https://docs.nextcloud.com/server/latest/admin_manual/configuration_server/activity_configuration.html#activities-in-groupfolders

How to get activity mails for groupfolders?

Edit: It does send mails, it just takes much longer to do so. Normally my NC checks for changes every 5 minutes and sends mails (at least for testing purposes) but this mail took 13 minutes or 3 cronjobs to be sent. Still a real fix (at least a checkbox in settings?) would be really useful. The drawbacks of the fix as mentioned in the documentation are not really that significant for most use cases I guess.

@nickvergessen
Copy link
Member

We are not going to add a checkbox. The config with dedicated documentation page is the real solution.

@maeno14
Copy link

maeno14 commented Jan 23, 2025

This should work as intended it should be defaulted to show activity in group folders to the members of the group allowed to view the folder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.