Skip to content

[backport stable34] Fix photosphere viewer not loading in NC33+ (#176)#179

Closed
R0Wi wants to merge 1 commit into
stable34from
backport/daf9cd0--to-stable34
Closed

[backport stable34] Fix photosphere viewer not loading in NC33+ (#176)#179
R0Wi wants to merge 1 commit into
stable34from
backport/daf9cd0--to-stable34

Conversation

@R0Wi
Copy link
Copy Markdown
Member

@R0Wi R0Wi commented May 26, 2026

Backport #176 to stable34

* Initial plan

* Fix XMP metadata serialization for NC33 WebDAV client compatibility

Agent-Logs-Url: https://github.com/nextcloud/files_photospheres/sessions/ddda8754-90c3-44ca-97c8-9703fbac9f52

Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>

* fix: XmpDataReader Test signature

* fix: Remove jQuery

* fix: NC34 compat

* Fix PHP controller interfaces
* Remove CSP for JS eval
* JS lib for modal
* Update nextcloud/files and adjust registering / callback code for frontend event handler accordingly

* Fix unit tests for NC34 API compatibility

Agent-Logs-Url: https://github.com/nextcloud/files_photospheres/sessions/d1f3fb6c-898e-47eb-8812-29e6a650a22c

Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>

* Apply php-cs-fixer changes

* fix: update filesAction.js.map

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>
Co-authored-by: Robin Windey <ro.windey@gmail.com>
Co-authored-by: Copilot <Copilot@users.noreply.github.com>
@R0Wi R0Wi changed the title [backport stable34] Backport PR 176 [backport stable34] Fix photosphere viewer not loading in NC33+ (#176) May 26, 2026
@R0Wi
Copy link
Copy Markdown
Member Author

R0Wi commented May 26, 2026

Waiting for stable34 to be ready

@R0Wi
Copy link
Copy Markdown
Member Author

R0Wi commented May 26, 2026

Will be picked with #177

@R0Wi R0Wi closed this May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants