-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
This commit improves performance by linking the current version of a record to its main stub, thereby making retrieval faster for records with lots of versions. The version retrieval functionality is retained for the purpose of historical view, which operates as it previously did. This commit also includes the addition of curval subfields to the y-axis of graphs (back-end only) which was needed to test some of the changes. It also includes sorting for grouping, which was needed for keeping the grouping tests working as before.
@droberts-ctrlo
droberts-ctrlo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Various files appear to be GH/Git incorrectly marking changes (as this was based off Dev) - I will keep these in, but they are marked with comments. Should they need to be removed, I shall do so on request.
One further change to push in order to remove changes that appear erroneous.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know what this is - was pulled through with my rebase into Dev
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what this is(?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't be here - not sure why it's included this when this change is already present
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure what this is
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure why this is here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs updating/changing - currently incorrect/invalid.
ee53b9e to
f8fb805
Compare
@pwlodarski-ctrlo
pwlodarski-ctrlo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR reviewed - comments attached.
@pwlodarski-ctrlo
pwlodarski-ctrlo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR reviewed - all good!
droberts-ctrlo
commented
Jun 8, 2026
This is now dependent on changes within #628 - will await this merge and update accordingly
No description provided.