Skip to content

[stable30] fix: Create download attribute when toggling checkbox - #52516

Merged
nfebe merged 2 commits into
stable30from
fix/stable30/create-download-attribute-if-missing
May 16, 2025
Merged

[stable30] fix: Create download attribute when toggling checkbox#52516
nfebe merged 2 commits into
stable30from
fix/stable30/create-download-attribute-if-missing

Conversation

@nfebe

Copy link
Copy Markdown
Contributor

Previously, toggling the checkbox did not create the 'download' attribute if it was missing, causing it to become unresponsive after a page reload. Now, setShareAttribute ensures the attribute is updated or created correctly.

Also a partial backport of : #50739

@nfebenfebe changed the title fix(files_sharing): Create download attribute when toggling checkbo…[stable30] fix(files_sharing): Create download attribute when toggling checkbo…Apr 28, 2025
@nfebenfebe changed the title [stable30] fix(files_sharing): Create download attribute when toggling checkbo…[stable30] fix: Create download attribute when toggling checkboxApr 28, 2025
@nfebenfebe added the 3. to review Waiting for reviews label Apr 28, 2025
@github-project-automationgithub-project-automationBot moved this to 🏗️ In progress in 📁 Files teamApr 28, 2025
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable29

@nfebe

Copy link
Copy Markdown
ContributorAuthor

/backport to stable28

@nfebe
nfebeforce-pushed the fix/stable30/create-download-attribute-if-missing branch from c0ff359 to 300c216CompareApril 28, 2025 23:37
@nfebe
nfebe enabled auto-merge April 28, 2025 23:37
@szaimen
szaimen removed their request for review April 29, 2025 08:29
@nfebe
nfebeforce-pushed the fix/stable30/create-download-attribute-if-missing branch from 300c216 to 250c072CompareApril 29, 2025 08:29
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/compile

…x if missing
Previously, toggling the checkbox did not create the 'download' attribute if it was missing,
causing it to become unresponsive after a page reload. Now, setShareAttribute ensures the
attribute is updated or created correctly.
Signed-off-by: nfebe <fenn25.fn@gmail.com>
@nfebe
nfebeforce-pushed the fix/stable30/create-download-attribute-if-missing branch from e4ddf65 to 6558dc9CompareMay 16, 2025 11:37
@nfebe

Copy link
Copy Markdown
ContributorAuthor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>

@susnuxsusnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

seems to work :)

@nextcloud-botnextcloud-bot mentioned this pull request Jun 4, 2025
11 tasks
@nextcloud-botnextcloud-bot mentioned this pull request Sep 18, 2025
@nextcloud-botnextcloud-bot mentioned this pull request Sep 25, 2025
2 tasks
@blizzzblizzz mentioned this pull request Oct 14, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: sharing

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

4 participants

@nfebe@susnux@artonge@nextcloud-command