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

allow user flows when the acting user is legitimate, but not its owner #18134

Merged
merged 5 commits into from
Nov 28, 2019

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Nov 27, 2019

for instance, when a sharee changes a file, the owner can act upon

To Dos:

  • expose the owner of the operation to act (e.g. when writing to conversations)

@blizzz blizzz added this to the Nextcloud 18 milestone Nov 27, 2019
@blizzz blizzz requested a review from juliusknorr November 27, 2019 13:02
@blizzz blizzz force-pushed the enh/noid/flow-extend-scope branch from ed6936e to a79ab62 Compare November 27, 2019 15:05
@blizzz blizzz marked this pull request as ready for review November 27, 2019 15:06
@blizzz blizzz force-pushed the enh/noid/flow-extend-scope branch from 3c81ed3 to 22f5fc8 Compare November 28, 2019 09:53
@blizzz blizzz requested a review from juliusknorr November 28, 2019 09:53
@blizzz
Copy link
Member Author

blizzz commented Nov 28, 2019

Tests also need #18143 it seems

@rullzer
Copy link
Member

rullzer commented Nov 28, 2019

Tests also need #18143 it seems

in

@juliusknorr
Copy link
Member

@blizzz I pushed another commit to set the file info in the rule matcher. That was missing when I tested it with the talk operation. Please give it a quick sanity check. Otherwise looks good to me and works as advertised.

Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

blizzz and others added 4 commits November 28, 2019 13:51
for instance, when a sharee changes a file, the owner can act upon

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
the File entity starts with a display text (used by talk)

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@blizzz blizzz force-pushed the enh/noid/flow-extend-scope branch from 901510e to 260f0fa Compare November 28, 2019 12:51
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzz
Copy link
Member Author

blizzz commented Nov 28, 2019

remaining failing tests are irrelevant to this PR

Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@rullzer rullzer merged commit e5c95ee into master Nov 28, 2019
@rullzer rullzer deleted the enh/noid/flow-extend-scope branch November 28, 2019 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants