Jquery upgrade#151
Merged
Merged
Conversation
Updated new jquery backwards compatible libraries and removed json2 as it's no longer needed
Owner
Author
|
Tested in a current online environment with edge Dev (latest as of today) all functionality was working with exception of azure mobile which I believe is due to it being deprecated, will remove that in a future PR. |
…DbOrgSettings/jquery1_11_0.js
…DbOrgSettings/jquery1_11_0.js.data.xml
…DbOrgSettings/json2.js.data.xml
…DbOrgSettings/json2.js
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request upgrades the jQuery library from version 1.11.0 to 3.7.1 and adds jQuery Migrate 3.6.0 to facilitate backward compatibility. The PR also removes the json2.js library, which is no longer needed as JSON functionality is natively supported in modern browsers.
Key changes:
- Upgraded jQuery from 1.11.0 to 3.7.1
- Added jQuery Migrate 3.6.0 for backward compatibility support
- Removed json2.js dependency and replaced
oJSON.stringify()calls with nativeJSON.stringify()
Reviewed changes
Copilot reviewed 9 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| orgDBOrgSettings.html | Updated script references to use new jQuery 3.7.1 and jQuery Migrate, removed json2.js reference, replaced oJSON.stringify() with JSON.stringify(), modified jQuery.noConflict() parameter |
| json2.js | Removed obsolete json2.js library file |
| json2.js.data.xml | Removed metadata file for json2.js web resource |
| jquery1_11_0.js.data.xml | Removed metadata file for old jQuery 1.11.0 web resource |
| jquery-3.7.1.min.js | Added new jQuery 3.7.1 minified library |
| jquery-3.7.1.min.js.data.xml | Added metadata file for jQuery 3.7.1 web resource |
| jquery-migrate-3.6.0.min.js | Added jQuery Migrate 3.6.0 for backward compatibility |
| jquery-migrate-3.6.0.min.js.data.xml | Added metadata file for jQuery Migrate web resource |
| Solution.xml | Updated version number, updated root components to reference new jQuery files, formatted XML elements |
| Relationships.xml | Removed empty relationships file |
| Customizations.xml | Updated with organization version and schema information |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
remove trailing white space
removed trailing spaces
removing spaces again
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added new updated / current jquery and recommended jquery migrate to bring it up to modern standards
Removed json2 as it's no longer needed