Skip to content

#ILEX-92 - Add Term#99

Merged
ddelpiano merged 11 commits intodevelfrom
feature/ILEX-92-post
Jun 11, 2025
Merged

#ILEX-92 - Add Term#99
ddelpiano merged 11 commits intodevelfrom
feature/ILEX-92-post

Conversation

@jrmartin
Copy link

@jrmartin jrmartin commented May 7, 2025

Summary
This PR allows doing a POST on olympian gods to submit a new entity. User must be login. The term is sent as a POST to the Olympian Gods endpoint /priv/entity-new, which is then follow with a response with redirect location. This PR also adds support for following 303 redirects returned during term creation. Axios intercepts these responses and fetches the redirected resource (e.g. .html, .jsonld, .ttl) through the Vite proxy to avoid CORS errors. Term IDs like TMP:000000057 or ILX:0622415 are extracted and displayed in the UI as part of the Add Term flow.

🔧 Changed Files Related to POST

mock/mutator/customClient.ts – Axios interceptor for 303 + redirect follow
vite.config.js – Proxy for dynamic tmp_* and ilx_* files with multiple extensions
src/api/endpoints/apiService.ts – createNewEntity() handles redirects and ID parsing
src/components/TermEditor/AddNewTermDialogContent.jsx – Integrated with new logic, shows formatted term ID
src/components/common/StatusStep.jsx + termStatusProps.js – Updated success/error UI handling

🔧 Other Changed Files
→ All updated to call getMatchTerms() with correct group and search term. Also a fix to retrieving raw data without using orval

src/components/SingleTermView/OverView/CustomizedTable.jsx
src/components/SingleTermView/OverView/OverView.jsx
src/components/SingleTermView/OverView/RawDataViewer.jsx
src/components/SingleTermView/OverView/PredicateGroupInput.jsx
src/components/SingleTermView/RequestMergeChanges.jsx

@jrmartin jrmartin changed the base branch from main to devel May 7, 2025 14:34
@jrmartin jrmartin requested a review from ddelpiano May 26, 2025 21:25
@jrmartin jrmartin marked this pull request as ready for review May 26, 2025 21:25
@ddelpiano
Copy link
Member

@jrmartin could you fix the linter issues? Thanks

@jrmartin
Copy link
Author

@jrmartin could you fix the linter issues? Thanks

@ddelpiano It's not an eslint issue, the github workflow is/was falling while fetching a library.

lint is fine locally and building image:

(base) jesusmartinez@ip-10-156-2-12 interlex % yarn run lint
yarn run v1.22.22
$ eslint . --ext js,jsx --report-unused-disable-directives --max-warnings 0
✨  Done in 3.21s.
(base) jesusmartinez@ip-10-156-2-12 interlex % 

Copy link
Member

@ddelpiano ddelpiano left a comment

Choose a reason for hiding this comment

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

lgtm

@ddelpiano ddelpiano merged commit e6a3ed1 into devel Jun 11, 2025
1 check 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