Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

feat: update magento-api package 1.1.0 -> 2.1.1 #17

Merged
merged 1 commit into from
Aug 18, 2023

Conversation

bartoszherba
Copy link
Collaborator

@bartoszherba bartoszherba commented Aug 17, 2023

To address the issue with potential session leak, I updated the magento-api package to the newest and applied all necessary changes to make the application work with the new package.

🔗 Linked issue

❓ Type of change

  • 📖 Documentation (updates to the documentation, readme or JSDoc annotations)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@bartoszherba bartoszherba requested review from a team August 17, 2023 12:06
Copy link

@Fifciu Fifciu left a comment

Choose a reason for hiding this comment

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

We need to also update the dedicated package.json for the middleware which is only for the workflow.

@bartoszherba bartoszherba force-pushed the feat/update-magento-api-package branch from e634846 to ed3bce5 Compare August 17, 2023 12:30
@bartoszherba bartoszherba requested a review from Fifciu August 17, 2023 12:30
@bartoszherba bartoszherba force-pushed the feat/update-magento-api-package branch from ed3bce5 to 899bcbc Compare August 18, 2023 08:40
@bartoszherba bartoszherba merged commit 12acb91 into main Aug 18, 2023
@sonarcloud
Copy link

sonarcloud bot commented Aug 18, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants