Skip to content

Conversation

@rullzer
Copy link
Member

@rullzer rullzer commented Apr 16, 2019

No description provided.

@rullzer rullzer added the 3. to review Waiting for reviews label Apr 16, 2019
@rullzer rullzer added this to the Nextcloud 17 milestone Apr 16, 2019
@rullzer rullzer force-pushed the enh/share_repair_step branch from 2b03829 to 093c308 Compare April 16, 2019 13:41
rullzer and others added 4 commits April 16, 2019 15:42
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
@rullzer
Copy link
Member Author

rullzer commented Apr 17, 2019

/backport to stable16

@rullzer
Copy link
Member Author

rullzer commented Apr 17, 2019

/backport to stable15

@rullzer
Copy link
Member Author

rullzer commented Apr 17, 2019

/backport to stable14

@rullzer rullzer merged commit 5382eec into master Apr 17, 2019
@backportbot-nextcloud
Copy link

The backport to stable16 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable15 failed. Please do this backport manually.

@rullzer rullzer deleted the enh/share_repair_step branch April 17, 2019 08:13
@backportbot-nextcloud
Copy link

The backport to stable14 failed. Please do this backport manually.


if ($notification->getSubject() === 'repair_exposing_links') {
$notification->setParsedSubject($l->t('Some of your link shares have been removed'));
$notification->setParsedMessage($l->t('Due to a security bug we had to remove some of your link shares. Please see the link for more information.'));
Copy link
Member

Choose a reason for hiding this comment

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

We got the feedback on this one that we in the future should properly communicate the actual removed links/files so that the user has the chance to recreate them manually.

Copy link
Collaborator

Choose a reason for hiding this comment

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

To add some more feedback: #15318 #15643 #16131 ;)

Copy link
Member

Choose a reason for hiding this comment

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

🙈 Too late I would say - for the next time we know it better and can improve on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants