Skip to content

feat(dep): SJIP-1608 upgrade axios and lodash - #1031

Merged
AltefrohneGaelle merged 6 commits into
devfrom
feat/sjip-1608
Sep 10, 2026
Merged

AltefrohneGaelle merged 6 commits into
devfrom
feat/sjip-1608

Conversation

@AltefrohneGaelle

@AltefrohneGaelle AltefrohneGaelle commented Sep 9, 2026 •

Copy link
Copy Markdown
Collaborator

feat(dep): upgrade axios and lodash

Description

Check dependabot PR and upgrade dependencies.

Acceptance Criterias

  • Upgrade axios
  • Upgrade lodash
  • Remove useless dependencies
  • No phantom dependency

@netlify

netlify Bot commented Sep 9, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for include-qa ready!

Name Link
🔨 Latest commit 6859817
🔍 Latest deploy log https://app.netlify.com/projects/include-qa/deploys/6aa1aad6ca64910008d3519f
😎 Deploy Preview https://deploy-preview-1031--include-qa.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

evans-g-crsj
evans-g-crsj previously approved these changes Sep 9, 2026

@evans-g-crsj evans-g-crsj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Fine like this.
You can check to remove lodash too trading it for in-place custom functions.

@AltefrohneGaelle

Copy link
Copy Markdown
Collaborator Author

Fine like this.
You can check to remove lodash too trading it for in-place custom functions.

I just checked, and the dependency is fully imported by ferlab-ui in any case, so there are no savings on the deployment bundle.

@AltefrohneGaelle
AltefrohneGaelle merged commit c60c830 into dev Sep 10, 2026
6 checks passed
@AltefrohneGaelle
AltefrohneGaelle deleted the feat/sjip-1608 branch September 10, 2026 12:20
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