-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/update apis to v1 #849
Conversation
446d3fc
to
8f1b815
Compare
Rebased and pushed, to see test results. |
8f1b815
to
6be2240
Compare
Had to update node, with new npm, same version as github actions, I get the same errors:
|
6be2240
to
b2f7e01
Compare
Looking at the changes made, I think it still needs something to be done about the routing, and checking that user is logged in. Assuming these were not there before. |
Headless run of the Cypress tests locally fail with login "│ ✖ login.spec.ts ". Locally even e2etest run one by one pass. Running 15 integration tests broke Chrome. |
b4d29ce
to
a133eee
Compare
The e2e tests to pass need the CSCfi/metadata-submitter#554 merged in the backed develop branch. Locally with that branch the tests passes. |
Done |
9734942
to
768c6a7
Compare
No tests fail for getting 400 from v1/schemas/datacite . |
I'm sorry, but I didn't understand. Do you mean that |
432e2a7
to
b12db04
Compare
Description
Add /v1 in API urls to adapt backend change.
Related issues
Fixes #826
Type of change
Changes Made
The frontend uses now following APIs
"/v1/objects"
"/v1/schemas"
"/v1/validate"
"/v1/submit"
"/v1/submissions"
"/v1/publish"
"/v1/drafts"
"/aai"
"/callback"
"/logout"
"/v1/users"
"/v1/templates"
Added a line about /v1 in API urls in CHANGELOG.
Rename cookie MTD_SESSION to AOIHTTP_SESSION in e2e login test.
Testing
Mentions