Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cool.html: append WOPISrc parameter to all JS, CSS & SVG loads #9034

Closed
wants to merge 2 commits into from

Conversation

Rash419
Copy link
Contributor

@Rash419 Rash419 commented May 10, 2024

Change-Id: I1dd3902403a93914a8ad03b24bed5524a59e5aca

  • Target version: master

@Rash419 Rash419 requested review from mmeeks and quikee May 10, 2024 17:31
Copy link
Contributor

@mmeeks mmeeks left a comment

Choose a reason for hiding this comment

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

Looks great to me - thanks Rash =) Did you check we clobber every URL by checking in the browser Network tab as things load ? =)

@Rash419
Copy link
Contributor Author

Rash419 commented May 13, 2024

Looks great to me - thanks Rash =) Did you check we clobber every URL by checking in the browser Network tab as things load ? =)

@mmeeks I did. Do icons url also needs to be appended with WOPISrc ?

@mmeeks
Copy link
Contributor

mmeeks commented May 14, 2024

Re: icons - yes, all URLs should have that.

@Rash419
Copy link
Contributor Author

Rash419 commented May 15, 2024

Re: icons - yes, all URLs should have that.

OK, done

@Rash419 Rash419 changed the title cool.html: append WOPISrc -or- RouteToken parameter to all JS & CSS loads cool.html: append WOPISrc parameter to all JS, CSS & SVG loads May 15, 2024
Signed-off-by: Rashesh Padia <[email protected]>
Change-Id: I1dd3902403a93914a8ad03b24bed5524a59e5aca
- use keyboard shortcut to insert comment

Signed-off-by: Rashesh Padia <[email protected]>
Change-Id: Ifa3dfd392d3ec6e62b896d34e19b1e9f5854d3c8
Copy link
Contributor

@mmeeks mmeeks left a comment

Choose a reason for hiding this comment

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

So - I think we should abandon this for now - I suspect this will break browser caching; and that we should come up with a better approach & design here.

@mmeeks
Copy link
Contributor

mmeeks commented May 18, 2024

Also a duplicate pull request it seems; lets abandon this one for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants