(refactor) Remove redundant "value" state derived from the "previous value" #306

gitcliff · 2024-06-05T07:13:14Z

Requirements

This PR has a title that briefly describes the work done including the ticket number. If there is a ticket, make sure your PR title includes a conventional commit label. See existing PR titles for inspiration.
My work conforms to the OpenMRS 3.0 Styleguide and design documentation.
My work includes tests or is validated by existing tests.

This PR is removing the redundant value state coming from the previous value across all components in the engine

samuelmale · 2024-06-05T08:20:05Z

@gitcliff can you ensure that the cleanup cuts across all components?

gitcliff · 2024-06-05T10:16:48Z

@gitcliff can you ensure that the cleanup cuts across all components?

@samuelmale i have added more cleanups
thanks

samuelmale · 2024-06-05T13:58:50Z

Can you also add a summary of what this PR does?

gitcliff · 2024-06-05T14:32:43Z

Can you also add a summary of what this PR does?

thanks ...
Done

gitcliff added 2 commits June 5, 2024 10:12

(fix):Removing value state

9f66010

(fix):Removing value state

9909dba

gitcliff requested a review from samuelmale June 5, 2024 07:16

gitcliff mentioned this pull request Jun 5, 2024

(fix)03-3338 :Refactor extraction of previous value to the section component. #303

Merged

3 tasks

samuelmale changed the title ~~(fix):Removing value state~~ (refactor) Remove redundant "value" state derived from the "previous value" Jun 5, 2024

gitcliff added 2 commits June 5, 2024 12:29

(fix):adding more clean ups of value state

ad7d3c6

(fix):adding more clean ups of value state

420668a

samuelmale approved these changes Jun 5, 2024

View reviewed changes

samuelmale merged commit 30e7a1f into openmrs:main Jun 5, 2024
4 checks passed