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

@nfebe nfebe commented Apr 28, 2025

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

@nfebe nfebe 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
@nfebe nfebe changed the title [stable30] fix(files_sharing): Create download attribute when toggling checkbo… [stable30] fix: Create download attribute when toggling checkbox Apr 28, 2025
@nfebe nfebe added the 3. to review Waiting for reviews label Apr 28, 2025
@github-project-automation github-project-automation Bot moved this to 🏗️ In progress in 📁 Files team Apr 28, 2025
@nfebe

nfebe commented Apr 28, 2025

Copy link
Copy Markdown
Contributor Author

/backport to stable29

@nfebe

nfebe commented Apr 28, 2025

Copy link
Copy Markdown
Contributor Author

/backport to stable28

@nfebe nfebe force-pushed the fix/stable30/create-download-attribute-if-missing branch from c0ff359 to 300c216 Compare April 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 nfebe force-pushed the fix/stable30/create-download-attribute-if-missing branch from 300c216 to 250c072 Compare April 29, 2025 08:29
@nfebe

nfebe commented Apr 29, 2025

Copy link
Copy Markdown
Contributor Author

/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 nfebe force-pushed the fix/stable30/create-download-attribute-if-missing branch from e4ddf65 to 6558dc9 Compare May 16, 2025 11:37
@nfebe

nfebe commented May 16, 2025

Copy link
Copy Markdown
Contributor Author

/compile

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

@susnux susnux 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 :)

@nfebe nfebe merged commit 5b1e955 into stable30 May 16, 2025
117 checks passed
@nfebe nfebe deleted the fix/stable30/create-download-attribute-if-missing branch May 16, 2025 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: 🏗️ In progress

Development

Successfully merging this pull request may close these issues.

4 participants