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

Remove duplicated TEIMP duration when showing diffs #1847

Open
Sepehr-Sobhani opened this issue Aug 9, 2023 · 0 comments
Open

Remove duplicated TEIMP duration when showing diffs #1847

Sepehr-Sobhani opened this issue Aug 9, 2023 · 0 comments
Labels
Tech Debt Technical Debt

Comments

@Sepehr-Sobhani
Copy link
Contributor

###Description of the Tech Debt###

When displaying the differences related to the TEIMP form and the TEIMP end date has been modified, the duration is displayed twice, resulting in a duplicate ID for the content suffix in the displayed diffs (as seen in the image below). To address this issue, a random ID generator has been added in the CustomDiffFields file
(id={id && `${id}-contentSuffix-${Math.random().toFixed(2)}`}). However, a better solution would be to prevent the duplicate duration from appearing in the diffs altogether. Additionally, an assertion needs to be added to verify the presence of this duration element in the diffs in the edit-project-revision.cy.js file, around line 239.

image.png

###Tech Debt Triage#
The purpose of our technical debt triage process is to analyze technical debt to determine risk level of the technical debt and the value in tackling that technical debt.

Risk Value Scoring:

Level Value
High 3
Medium 2
Low 1
Technical Debt - Risk Types Level Value
Business Area Risk - Risk of business area visibility / damage to user experience 1
Developer Fault Risk - How likely will this tech debt cause a future error related to coding on top of it 1
System Fault Risk - Risk of system errors or application downtime 1
Time Scale Risk - Compound risk effect if left alone. How much more difficult to fix or dangerous will this become over time? 1
Time Sink Risk - How much will this tech debt slow the development process down 1
TOTAL SCORE: 5
@Sepehr-Sobhani Sepehr-Sobhani added the Tech Debt Technical Debt label Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tech Debt Technical Debt
Projects
None yet
Development

No branches or pull requests

1 participant