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

Spike building a licence history page #1167

Closed
wants to merge 10 commits into from
Closed

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented Jul 5, 2024

https://eaflood.atlassian.net/browse/WATER-4541

We've started talking about the NALD 'mod log'. Our current understanding is that whenever a user creates a new return, licence, or charge version NALD pops up an 'event' where the user is expected to enter a reason and a note. Essentially, why the change is being made.

They then have reports that combine these events. They use these to ensure the 3 legs of water abstraction are in sync. (They often use the metaphor of a stool, with licence (abs), returns and charging the 3 legs).

When viewed as a report, this essentially just seems to be what other systems would call history i.e. the history of changes to a licence.

There is a concern that moving away from the NALD also means losing the 'mod log' and therefore how would they ensure things are in sync.

This 'spike' is to build an example licence history page that combines licence, return and charge versions into one place to see if any of these concerns can be alleviated.

water-4566-initial-spike

https://eaflood.atlassian.net/browse/WATER-4541

We've started talking the NALD 'mod log'. Our current understanding is that whenever a user creates a new return, licence or charge version NALD pops up an 'event' where the user is expected to enter a reason and a note. Essentially, why the change is being made.

They then have reports that combine these events. They use these to ensure the 3 legs of water abstraction are in sync. (They often use the metaphor of a stool, with licence (abs), returns and charging the 3 legs).

When viewed as a report, this essentially just seems to be what other systems would call history i.e. the history of changes to a licence.

There is a concern that moving away from the NALD also means losing the 'mod log' and therefore how would they ensure things are in sync.

This 'spike' is to build an example licence history page that combines licence, return and charge versions into one place to see if any of these concerns can be alleviated.
@Cruikshanks Cruikshanks added the do not merge Used for spikes and experiments label Jul 5, 2024
@Cruikshanks Cruikshanks self-assigned this Jul 5, 2024
Sort by date desc. Where two entries have the same date then go by order of type

- return version
- charge version
- licence version

We read the history from latest (top) to earliest (bottom). Where we have matching dates we go with an assumed order for version entry; licence then charge then return version.
After a quick consult with our designer agreed it would be better to split the notes from the return version and the mod log rather than merge them into a single note.
We now have a table so this is redundant. Will update properly later
@Cruikshanks
Copy link
Member Author

This has served its purpose. We now have #1182 approved and ready to be merged. So, bye-bye spike!

@Cruikshanks Cruikshanks deleted the spike-licence-history branch August 29, 2024 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge Used for spikes and experiments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant