chore: use main IDs transaction_id and staging_entry_id for URL routing and data fetching
#3847
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Type of Change
Description
This PR updates the ReconEngine URLs and data fetching logic to use main IDs (
transaction_idandstaging_entry_id) instead of unique IDs. Due to versioning requirements, the application now routes and fetches data using the main identifiers, which allows proper handling of versioned entries.Key Changes:
transaction_idinstead of uniqueidstaging_entry_idinstead of uniqueidgetShowLinkupdated for proper navigationMotivation and Context
With versioning support in the ReconEngine, each transaction and transformed entry can have multiple versions. Using the unique
idfield causes issues when navigating and fetching data because it points to a specific version rather than the main entity. Byswitching to
transaction_idandstaging_entry_id(the main identifiers), we can:Fixes: #3846
How did you test it?
Locally
Where to test it?
Checklist
npm run re:build