ContextPath Prefixed for multiple calls via $.ajax/.getJson & more #65
Conversation
friendlymahi
commented
Dec 9, 2023
friendlymahi
commented
- Fixes Issue BrightCove Dialogs & Admin UI doesn't load when context path is enabled #64
|
@hngrylobster - Requesting you to review these changes and advise for any addl fixes. We recently upgraded our AEM to v6.5 SP17 and long due on upgrading Brightcove. With this fix we'll be clean without having to make any changes on clientlibs and manage your code copy on our end. Thanks in advance ! |
|
@hngrylobster - Happy New Year ! We wanted to know if you intend to review and advise on these fixes so we can communicate to our leadership team on the next upgrade/version bump of brightcove in our org. Thanks !! |
|
@friendlymahi I asked a question on this to possibly resolve. Also if this is for on prem please cancel this pr do changes on your own branch from release/6.0.5 and do your pr to merge into this branch. If clould do the changes from release/6.1.2-cloud. We will be working on combining these branches in the future so that one single master will work. Thank you |
|
@dcolvinbc - sure. Will send a new pr as this is for onprem. Also shall I send the PR back to release/6.0.5 itself or master? ! |
|
The PR should be to merge to release/6.0.5. |
|
Thanks again for taking time to respond. I will send a PR today. |
|
Closing this PR in favor of the new one. Thanks ! |
