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

WIP: Update and refactor LinkToEdit component #445

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

alexlipovka
Copy link
Collaborator

  • Moved to src/components
  • Added new AidboxQuery to get latest provenance for resource
  • Call request for provenance within LinkToEdit

Motivation: reduce request for provenances with _revinclude when it's not needed, simplify and unify LinkToEdit component, which was defined twice

@alexlipovka alexlipovka requested a review from ir4y February 4, 2025 08:45
@alexlipovka alexlipovka changed the title Update and refactor LinkToEdit component WIP: Update and refactor LinkToEdit component Feb 4, 2025
@alexlipovka alexlipovka changed the title WIP: Update and refactor LinkToEdit component Update and refactor LinkToEdit component Feb 4, 2025
@alexlipovka alexlipovka changed the title Update and refactor LinkToEdit component WIP: Update and refactor LinkToEdit component Feb 4, 2025
@alexlipovka alexlipovka force-pushed the update-link-to-edit-auto-provenance branch 3 times, most recently from 351182a to ec56f20 Compare February 12, 2025 07:50
@alexlipovka alexlipovka force-pushed the update-link-to-edit-auto-provenance branch 2 times, most recently from d208f33 to b22596b Compare February 24, 2025 17:27
- Uses context to get custom path extraction
- Searches through all targets within provenance
- Always uses latest provenance
@alexlipovka alexlipovka force-pushed the update-link-to-edit-auto-provenance branch from b22596b to 31d3530 Compare February 25, 2025 06:03
@ir4y ir4y merged commit 69ff00c into master Feb 25, 2025
5 checks passed
@ir4y ir4y deleted the update-link-to-edit-auto-provenance branch February 25, 2025 09:13
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