Skip to content

Dependabot multi 9faa0f6d25 fix#1738

Open
Geoffsc wants to merge 4 commits intoUniversalViewer:devfrom
Geoffsc:dependabot-multi-9faa0f6d25-fix
Open

Dependabot multi 9faa0f6d25 fix#1738
Geoffsc wants to merge 4 commits intoUniversalViewer:devfrom
Geoffsc:dependabot-multi-9faa0f6d25-fix

Conversation

@Geoffsc
Copy link
Copy Markdown
Contributor

@Geoffsc Geoffsc commented Apr 21, 2026

To fix the server not starting issue as a result of dependabot's jquery upgrade to 4.0.0:

  • Updated jquery alias in webpack-dev-server.js to use path.resolve
  • Upgraded jsviews to the latest version which handles the removal of jQuery.isFunction

dependabot Bot and others added 2 commits March 9, 2026 00:52
Bumps [jquery](https://github.com/jquery/jquery) and [@types/jquery](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/jquery). These dependencies needed to be updated together.

Updates `jquery` from 3.5.0 to 4.0.0
- [Release notes](https://github.com/jquery/jquery/releases)
- [Changelog](https://github.com/jquery/jquery/blob/main/changelog.md)
- [Commits](jquery/jquery@3.5.0...4.0.0)

Updates `@types/jquery` from 3.5.14 to 4.0.0
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/jquery)

---
updated-dependencies:
- dependency-name: jquery
  dependency-version: 4.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
- dependency-name: "@types/jquery"
  dependency-version: 4.0.0
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
universalviewer Ready Ready Preview, Comment Apr 22, 2026 1:12pm

Request Review

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 21, 2026

@Geoffsc is attempting to deploy a commit to the Universal Viewer Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown
Contributor

@demiankatz demiankatz left a comment

Choose a reason for hiding this comment

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

Thanks, @Geoffsc, this is great progress!

I notice that the build is failing; I suspect that may be due to configuration changes needed in the regular webpack configuration (to parallel the changes in the dev server config). If you need help figuring that out, maybe we can pair on this after tomorrow's standup if time permits!

@Geoffsc
Copy link
Copy Markdown
Contributor Author

Geoffsc commented Apr 21, 2026

@demiankatz thank you, I believe it should be fixed now. All the tests are passing and npm run dev still works fine!

@demiankatz
Copy link
Copy Markdown
Contributor

Thanks, @Geoffsc, this does indeed look very promising! We'll probably want to test all of the extensions to be sure there aren't any gaps we're missing, but this is excellent progress. Maybe we can discuss at standup tomorrow what kind of testing we need in order to feel confident enough to merge this.

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