Skip to content

fix: #8358 migrate to @nexcloud/viewer library - #8685

Open
smarinier wants to merge 1 commit into
nextcloud:mainfrom
smarinier:fix/8358_migrate_off_the_oca_viewer_nc_36
Open

smarinier wants to merge 1 commit into
nextcloud:mainfrom
smarinier:fix/8358_migrate_off_the_oca_viewer_nc_36

Conversation

@smarinier

@smarinier smarinier commented Nov 6, 2026 •

Copy link
Copy Markdown

Summary

Attachment use old global OCA.Viewer and crashes on NC36 (window.OCA.Viewer.availableHandlers.map)

TODO

  • [ ]

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
  • [not required] Documentation (manuals or wiki) has been updated or is not required

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI (N/A)

@blizzz blizzz added 3. to review javascript Pull requests that update Javascript code dependencies: javascript labels Nov 6, 2026
@blizzz

blizzz commented Nov 6, 2026

Copy link
Copy Markdown
Member

Thanks for your contribution!

Please add the Assisted-by trailer to the commit as per policy: https://github.com/nextcloud/.github/blob/master/AI_POLICY.md#disclosure

Assisted-by: ClaudeCode:Opus 5.5

Signed-off-by: Sebastien Marinier <seb@smarinier.net>
@smarinier
smarinier force-pushed the fix/8358_migrate_off_the_oca_viewer_nc_36 branch from 4fe6bb5 to 12f3b96 Compare November 6, 2026 17:23

This branch has not been deployed

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

Labels

3. to review dependencies: javascript javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate off the OCA.Viewer global before Nextcloud 36

2 participants