Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: downgrade axios version in order to improve compatibility #1

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

andrey-canon
Copy link
Collaborator

Description

The target version (open-release/palm.nelp) is a copy of this branch which is necessary to use the frontend-platform version that allows to inject css styles, however the target version includes a the axios 1.4.0 version which is not compatible with the axios-mock-adapter library, so when this version is included in packages like frontend-app-learning the axios version is updated and the test starts to fail

Before

image

image

After

image

Copy link

@johanseto johanseto left a comment

Choose a reason for hiding this comment

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

Seem paragon-alpha is not compatible in tests due dependencies.

By other hand I thought this was related this error, but no.
image
That error seems related to other service refused connection.
image

@andrey-canon
Copy link
Collaborator Author

andrey-canon commented Jan 17, 2024

Seem paragon-alpha is not compatible in tests due dependencies.

By other hand I thought this was related this error, but no.
image
That error seems related to other service refused connection.
image

The test error is something that is happening also in the target version, as I mentioned in the description that comes from the branch ags/2331and that includes that error

On the other hand the other error is because you need to set the EXAMS_BASE_URL, you need to set that to an empty string.

@andrey-canon andrey-canon force-pushed the open-release/palm.nelp branch from 84cbeec to 84c6d76 Compare January 18, 2024 20:57
@johanseto johanseto self-requested a review January 18, 2024 21:16
@andrey-canon andrey-canon merged commit 0df7fbc into open-release/palm.nelp Jan 18, 2024
3 checks passed
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