Enhance suffix unit conversion deletion with dependency checks#1561

Open

hmhngx wants to merge 319 commits intoOpenEnergyDashboard:developmentfrom

hmhngx:deletion-update

This pull request is big! We're only showing the most recent 250 commits

Commits on Dec 11, 2025

adding test LR4

TVW96
authored and
hmhngx
committed
fix type error

TVW96
authored and
hmhngx
committed
remove line 47

TVW96
authored and
hmhngx
committed
add expected values csv

TVW96
authored and
hmhngx
committed
Update readingsLineMeterRangeQuantity.js

TVW96
authored and
hmhngx
committed
Test Case LR2

lukeckk
authored and
hmhngx
committed
updated file path

lukeckk
authored and
hmhngx
committed
removed comment

lukeckk
authored and
hmhngx
committed
Made the components, and the lint stop complaining. Next up update where it is used

TheREALNguyenAnh
authored and
hmhngx
committed
Seems to be working

TheREALNguyenAnh
authored and
hmhngx
committed
remove container

TheREALNguyenAnh
authored and
hmhngx
committed
Mozilla Public License Header

TheREALNguyenAnh
authored and
hmhngx
committed
Cleaning up some comments

TheREALNguyenAnh
authored and
hmhngx
committed
Add a check for infinity in calculate change

TheREALNguyenAnh
authored and
hmhngx
committed
Rename CompareChart to CompareBar

TheREALNguyenAnh
authored and
hmhngx
committed
Some comments and cleaning up

TheREALNguyenAnh
authored and
hmhngx
committed
Empty lines

TheREALNguyenAnh
authored and
hmhngx
committed
fix up compare bar icons

huss
authored and
hmhngx
committed
add plotly icons to compare line

huss
authored and
hmhngx
committed
Trying out half bound.

TheREALNguyenAnh
authored and
hmhngx
committed
A bit more about half bound

TheREALNguyenAnh
authored and
hmhngx
committed
DONE WITH LOGIC

TheREALNguyenAnh
authored and
hmhngx
committed
JSDoc and TypeScript taken care of

TheREALNguyenAnh
authored and
hmhngx
committed
Fix up the timezone problem

TheREALNguyenAnh
authored and
hmhngx
committed
Copy the initialXAxis setup to BarGraph and some messing arround with ChartDataSelectComponent

TheREALNguyenAnh
authored and
hmhngx
committed
Finished with buttons and things

TheREALNguyenAnh
authored and
hmhngx
committed
Put back the comments

TheREALNguyenAnh
authored and
hmhngx
committed
MPL headers added

TheREALNguyenAnh
authored and
hmhngx
committed
Remove dispatch

TheREALNguyenAnh
authored and
hmhngx
committed
Blank lines and some formatting

TheREALNguyenAnh
authored and
hmhngx
committed
Refactor code so that it has one returns

TheREALNguyenAnh
authored and
hmhngx
committed
More refactoring

TheREALNguyenAnh
authored and
hmhngx
committed
Address the history problem. I did it by exclude setInitialXAxis in the isHistoryTrigger and combine some actions into one.

TheREALNguyenAnh
authored and
hmhngx
committed
Removed Clear Date Range button cause the original X works after the fix to history

TheREALNguyenAnh
authored and
hmhngx
committed
Change around the bounded of More Options

TheREALNguyenAnh
authored and
hmhngx
committed
created SimpleUnsavedWarningComponent that is displayed on Edit Meters page; updated Edit Meters to have a boolean value that updates whenever there is a change

aduques
authored and
hmhngx
committed
implemented the Simple Unsaved Warning on all relevant edit/create pages; added boolean statements that are updated whenever a value is updated on relevant pages; reorganized the Maps page to work …

aduques
authored and
hmhngx
committed
updated problem on CSV pages that prevented users from interacting with the page after clicking on the unsaved warning's cancel button.

aduques
authored and
hmhngx
committed
make noscript warning nicer

huss
authored and
hmhngx
committed
make index.html use tab indenting

huss
authored and
hmhngx
committed
Update VSCode launch config to support debugging client and server code

evan-carey
authored and
hmhngx
committed
Add launch config for Firefox debugger

evan-carey
authored and
hmhngx
committed
made the unsaved warning's save button optional + setUnsavedWarning is now set by using canSave + setUnsavedWarning no longer added in each instance + fixed blank lines between imports (commented o…

aduques
authored and
hmhngx
committed
removed comments related to implementing useNavigator + reverted changes made on MapsDetailComponent

aduques
authored and
hmhngx
committed
added test case LR5

pazcharlee
authored and
hmhngx
committed
fix up code per comment in PR

huss
authored and
hmhngx
committed
Refactor unit and conversion refresh flow to align with centralized refreshing

shaikhibrahim2000
authored and
hmhngx
committed
Fix: Refresh now properly refetches Units to show suffix conversions

shaikhibrahim2000
authored and
hmhngx
committed
Fix: Refresh now properly refetches Units to show suffix conversions

shaikhibrahim2000
authored and
hmhngx
committed
Fixed linting issues

shaikhibrahim2000
authored and
hmhngx
committed
Code formatting

shaikhibrahim2000
authored and
hmhngx
committed
fixed cancel behavior for CSV pages + removed the canSave + removed the valid checks on all related modals (except Edit/CreateUnitModal) (saving issue on actual modal still needs to be addressed)

aduques
authored and
hmhngx
committed
added canSave again + added validChange that checks for required fields + updated SimpleUnsavedWarningComponent to include disabled + used canSave to disable save all button on create pages when ne…

aduques
authored and
hmhngx
committed
Added authentication middleware to conversions and conversionArray routes

rayan-hamid
authored and
hmhngx
committed
Patch access control for route files groups, login, logs, maps, meters, obvious, and preferences

misantiago99
authored and
hmhngx
committed
Applying fixes that Steve suggested. Reverted docker.compose.yml and package-lock.js by checking out from development branch and overriding anything that I could have done to those files.

misantiago99
authored and
hmhngx
committed
Merge remote-tracking branch 'origin/Calvin-patches' into Rayan-patches

rayan-hamid
authored and
hmhngx
committed
Pulled in updates from Miguel and Calvin: maps.js, login.js, package.json, readings.js, and units.js

rayan-hamid
authored and
hmhngx
committed
Resolved unitReadings.js conflict by using Calvin's updated version

rayan-hamid
authored and
hmhngx
committed
Resolved all reviewer feedback: route descriptions, validation schemas, formatting

rayan-hamid
authored and
hmhngx
committed
Updated route descriptions and authorization roles in users.js and conversionArray.js per PR feedback

rayan-hamid
authored and
hmhngx
committed
This changes several items in routes:

huss
authored and
hmhngx
committed
fix undesired changes to unit.js route

huss
authored and
hmhngx
committed
address two PR comments

huss
authored and
hmhngx
committed
minor edit from reading

huss
authored and
hmhngx
committed
configured cy.log to output to terminal when running in headless mode

KaylaCrush
authored and
hmhngx
committed
fixed indentation and semicolons

KaylaCrush
authored and
hmhngx
committed
fixed syntax issue

KaylaCrush
authored and
hmhngx
committed
Added ui test with logging example

KaylaCrush
authored and
hmhngx
committed
quick change to tabs for formatting

huss
authored and
hmhngx
committed
fix middleware to allow test login

huss
authored and
hmhngx
committed
fixed problem of not all fields being considered for unsaved changes + fixed problem of slope/intercept 0 not saving + added variable to manage conversionState + separated the submit logic for hand…

aduques
authored and
hmhngx
committed
updated comments + updated editMade logic + fixed issue of save button not being in sync between the unsaved warning and modal + CSV files now considered in editMade + removed deepMeterChanges

aduques
authored and
hmhngx
committed
updated comments + added disabled save buttons on edit pages

aduques
authored and
hmhngx
committed
implemented 'Leave' functionality to the unsaved warning in CSV pages + implemented unsaved warnings in edit/create user pages

aduques
authored and
hmhngx
committed
modified editMade on EditUser + removed redundant code + added commented out code for samePassword (will remove)

aduques
authored and
hmhngx
committed
Add test cases for all LogMsg routes in logs.js

omarraf
authored and
hmhngx
committed
enhance route tests with insert and verify functionality

omarraf
authored and
hmhngx
committed
add MPL license

omarraf
authored and
hmhngx
committed
specify log type in message

omarraf
authored and
hmhngx
committed
fix: time interval, remove weak filtering, add error tests

omarraf
authored and
hmhngx
committed
format document and add improve no logs match filters test

omarraf
authored and
hmhngx
committed
add proper time intervals

omarraf
authored and
hmhngx
committed
convert indentation to tabs

omarraf
authored and
hmhngx
committed
add simpler test for info log query

omarraf
authored and
hmhngx
committed
clean up comments

omarraf
authored and
hmhngx
committed
improve warn and error log tests

omarraf
authored and
hmhngx
committed
Change ways to keep track what is group and what is meter

TheREALNguyenAnh
authored and
hmhngx
committed
Merge complete. Formatting and labeling.

TheREALNguyenAnh
authored and
hmhngx
committed
Change to sorting

TheREALNguyenAnh
authored and
hmhngx
committed
Done! Going to need to change tooltips and Label first

TheREALNguyenAnh
authored and
hmhngx
committed
cleanup

TheREALNguyenAnh
authored and
hmhngx
committed
change the G and the M

TheREALNguyenAnh
authored and
hmhngx
committed
This version does not have the random deselect bug

TheREALNguyenAnh
authored and
hmhngx
committed
Done with the merging of meters and group

TheREALNguyenAnh
authored and
hmhngx
committed
Some clean up

TheREALNguyenAnh
authored and
hmhngx
committed
clearring 3d works properly

TheREALNguyenAnh
authored and
hmhngx
committed
Done with threeD logic

TheREALNguyenAnh
authored and
hmhngx
committed
Some fixes to formatting and commenting. Add the superscript to the showup label. Need to see if I can add it to the actual Chart Labeling

TheREALNguyenAnh
authored and
hmhngx
committed
Some formatting

TheREALNguyenAnh
authored and
hmhngx
committed
fix up merge conflict

TheREALNguyenAnh
authored and
hmhngx
committed
Some CSS and styling and adding the superscript to the label of radar chart.

TheREALNguyenAnh
authored and
hmhngx
committed
TooltipHelp with datasources

TheREALNguyenAnh
authored and
hmhngx
committed
Add the superscript M and G to map hover

TheREALNguyenAnh
authored and
hmhngx
committed
retrigger the github action

TheREALNguyenAnh
authored and
hmhngx
committed
things

TheREALNguyenAnh
authored and
hmhngx
committed
updated comments + rearranged variables to top of page + removed commented out code + fixed 'Cancel' on edit modal

aduques
authored and
hmhngx
committed
added back handleClose() to edit user as it was accidentally removed in previous commit

aduques
authored and
hmhngx
committed
addressed issue where user cannot save from unsaved warning page in edit user

aduques
authored and
hmhngx
committed
added login/logout and dropdown menu tests to the general ui tests

KaylaCrush
authored and
hmhngx
committed
generalized login/logout tests to both desktop and mobile viewports

KaylaCrush
authored and
hmhngx
committed
Moved login/logout tests to separate file

KaylaCrush
authored and
hmhngx
committed
Moved login/logout tests to separate file

KaylaCrush
authored and
hmhngx
committed
Got desktop version of login/logout working properly

KaylaCrush
authored and
hmhngx
committed
got mobile tests working alsogit status

KaylaCrush
authored and
hmhngx
committed
Fixed syntax issues (spaces to tabs, added a couple missing semicolons)

KaylaCrush
authored and
hmhngx
committed
Added header to loginLogoutTest.cy.ts

KaylaCrush
authored and
hmhngx
committed
fix general_ui.cy.ts

KaylaCrush
authored and
hmhngx
committed
Added language translation support

KaylaCrush
authored and
hmhngx
committed
Working on Stuff. Route and API added. Conversions typescript not final. Adding to EditConversionModalComponent next

TheREALNguyenAnh
authored and
hmhngx
committed
API connected. Bug on server side

TheREALNguyenAnh
authored and
hmhngx
committed
Lost units protections

TheREALNguyenAnh
authored and
hmhngx
committed
Adding back checkState()

TheREALNguyenAnh
authored and
hmhngx
committed
Done with the unit to unit. Might have to check for orphan unit

TheREALNguyenAnh
authored and
hmhngx
committed
Done with unit to unit case

TheREALNguyenAnh
authored and
hmhngx
committed
Finished

TheREALNguyenAnh
authored and
hmhngx
committed
Cik Applied. Saved 500 ms per call to SimulateDelete roughly.

TheREALNguyenAnh
authored and
hmhngx
committed
Got the time to sub 1 second

TheREALNguyenAnh
authored and
hmhngx
committed
Frontend formatting and Internationalization

TheREALNguyenAnh
authored and
hmhngx
committed
Things

TheREALNguyenAnh
authored and
hmhngx
committed
Things

TheREALNguyenAnh
authored and
hmhngx
committed
Got new conversion affected

TheREALNguyenAnh
authored and
hmhngx
committed
Some work with formatting. CSS is next for bolding

TheREALNguyenAnh
authored and
hmhngx
committed
Done with the CSS

TheREALNguyenAnh
authored and
hmhngx
committed
Some cleaning up

TheREALNguyenAnh
authored and
hmhngx
committed
intersectSets

TheREALNguyenAnh
authored and
hmhngx
committed
done with default graphic unit warning message

TheREALNguyenAnh
authored and
hmhngx
committed
Done with changing of no unit

TheREALNguyenAnh
authored and
hmhngx
committed
finished with moving the simulate delete logic to a service file

TheREALNguyenAnh
authored and
hmhngx
committed
General addressing of formatting

TheREALNguyenAnh
authored and
hmhngx
committed
Some refactor on the conversion simulation

TheREALNguyenAnh
authored and
hmhngx
committed
Now run simulate delete on meter unit as source as well

TheREALNguyenAnh
authored and
hmhngx
committed
omit using lodash

TheREALNguyenAnh
authored and
hmhngx
committed
typescript

TheREALNguyenAnh
authored and
hmhngx
committed
middleware

TheREALNguyenAnh
authored and
hmhngx
committed
headers

TheREALNguyenAnh
authored and
hmhngx
committed
Change compatible unit

TheREALNguyenAnh
authored and
hmhngx
committed
Mutation into query

TheREALNguyenAnh
authored and
hmhngx
committed
Simulate Mutation into query

TheREALNguyenAnh
authored and
hmhngx
committed
Promise packaged together

TheREALNguyenAnh
authored and
hmhngx
committed
move the delete and updating default_graphic_unit into a transaction in the backend

TheREALNguyenAnh
authored and
hmhngx
committed
Done with the new routes

TheREALNguyenAnh
authored and
hmhngx
committed
turn the current delete into delete that update default graphic unit

TheREALNguyenAnh
authored and
hmhngx
committed
invalidate meters data and group data incase default graphic unit is changed

TheREALNguyenAnh
authored and
hmhngx
committed
done with this so far

TheREALNguyenAnh
authored and
hmhngx
committed
fix up!

TheREALNguyenAnh
authored and
hmhngx
committed
Added test case LR7

HienXuanPham
authored and
hmhngx
committed
remove only

HienXuanPham
authored and
hmhngx
committed
LR3 testing implemented

jkwon100
authored and
hmhngx
committed
formatting corrected

jkwon100
authored and
hmhngx
committed
updated comment in EditUserModalComponent

aduques
authored and
hmhngx
committed
BSUPipeline work + fix

huss
authored and
hmhngx
committed
new disabled check choice

huss
authored and
hmhngx
committed
add warn on cumulative reset to routes/pages

huss
authored and
hmhngx
committed
solidify pipeline

huss
authored and
hmhngx
committed
add feedback feature for username, password, role selecting, confirming password when creating, editing user
hmhngx
committed
the invaldid property when editing role of user is unnecessary
hmhngx
committed
logging changes

huss
authored and
hmhngx
committed
try to only test failing tests

huss
authored and
hmhngx
committed
hack to remove TZ in CSV

huss
authored and
hmhngx
committed
try different TZ

huss
authored and
hmhngx
committed
new TZ on other test

huss
authored and
hmhngx
committed
put back all pipeline test

huss
authored and
hmhngx
committed
enable all tests

huss
authored and
hmhngx
committed
first implementation of CG17

authored and
hmhngx
committed
removed comment line

ehhdyyy
authored and
hmhngx
committed
disable meter from menu options if the unit is used in any conversion as source or destination

authored and
hmhngx
committed
add semicolon

authored and
hmhngx
committed
fix bar graph's bar labels not switching language issue#1506

authored and
hmhngx
committed
Added psuedocode for test case CG20 - Team I

fluffylenny
authored and
hmhngx
committed
Implemented some of the pseudocode into actual code

tazmeenahhmed
authored and
hmhngx
committed
Added meterDatakWOther to meterkWGroups for CG20

fluffylenny
authored and
hmhngx
committed
Fixing merge conflicts

tazmeenahhmed
authored and
hmhngx
committed
Cleaned up comments

authored and
hmhngx
committed
Removed 'Add CG20 here' comment and spaces below it

tazmeenahhmed
authored and
hmhngx
committed
made test

JazzJE
authored and
hmhngx
committed
fixed test; need to clean print statements

JazzJE
authored and
hmhngx
committed
added bg21 text to test

JazzJE
authored and
hmhngx
committed
cleaned up diff

RussianGravy
authored and
hmhngx
committed
removed print statements, reverted changes

RussianGravy
authored and
hmhngx
committed
fixed parsed csv file name and added expected csv file

JazzJE
authored and
hmhngx
committed
docs: Update README to explain learning sandbox fork

aldungo
authored and
hmhngx
committed
docs: Add comprehensive student workflow documentation

aldungo
authored and
hmhngx
committed
docs: Add project management and team coordination

aldungo
authored and
hmhngx
committed
ci: Add GitHub automation for test case tracking

aldungo
authored and
hmhngx
committed
docs: Add tool setup guides and mentor resources

aldungo
authored and
hmhngx
committed
docs: Add student welcome message

aldungo
authored and
hmhngx
committed
added function declaration for LR14

stockingstocker
authored and
hmhngx
committed
declared units u6, u7, u8, in LR14

stockingstocker
authored and
hmhngx
committed
Created array of conversions c5, c7 in readingsLineMeterRangeRaw.js

vyvy5555
authored and
hmhngx
committed
added expected readings file for test LR14

stockingstocker
authored and
hmhngx
committed
added some comments and adjusted some variables for units

stockingstocker
authored and
hmhngx
committed
added comments for next steps of LR14

stockingstocker
authored and
hmhngx
committed
added test number LR14 to mocha.it() and fixed typo in unitF

stockingstocker
authored and
hmhngx
committed
added the CG9 feature

AndrewTruong
authored and
hmhngx
committed
edited

AndrewTruong
authored and
hmhngx
committed
revert to original docker.yml file

AndrewTruong
authored and
hmhngx
committed
Revert to the original version

AndrewTr0612
authored and
hmhngx
committed
Commented out the line 24 in docker-compose.yml

authored and
hmhngx
committed
Fix entrypoint formatting in docker-compose.yml

AndrewTr0612
authored and
hmhngx
committed
Remove CG9 comment from readingsCompareGroupQuantity test

AndrewTr0612
authored and
hmhngx
committed
Revert to the original version of docker-compose.yml

AndrewTr0612
authored and
hmhngx
committed
remove blank line change in file

huss
authored and
hmhngx
committed
Update .gitattributes

Zachary-Squires
authored and
hmhngx
committed
turned off autorange for line chart, fixed some bugs from previous PR

MatthewTran22
authored and
hmhngx
committed
fixed sidebar range issue, auto adjusts to lowest x date value to highest date x value

MatthewTran22
authored and
hmhngx
committed
removed unused depedencies from LineChart file

MatthewTran22
authored and
hmhngx
committed
changed the naming, there is already a minX and maxX

MatthewTran22
authored and
hmhngx
committed
fixed styling issues

MatthewTran22
authored and
hmhngx
committed
removed trailing space

MatthewTran22
authored and
hmhngx
committed
addressed the first 3 comments from the PR, looking into the range slider

MatthewTran22
authored and
hmhngx
committed
possible solution to range slider issue, specified the x range with the min and max date

MatthewTran22
authored and
hmhngx
committed
styling fix

MatthewTran22
authored and
hmhngx
committed
linting fix

MatthewTran22
authored and
hmhngx
committed
added meter variable, implemented LR14 test, adjusted comments

authored and
hmhngx
committed
moved unit variable declarations into array initialization

stockingstocker
authored and
hmhngx
committed
removed added mentor guidance files

stockingstocker
authored and
hmhngx
committed
remove added mentor guidance files

stockingstocker
authored and
hmhngx
committed
revert accidental edit of readme file

stockingstocker
authored and
hmhngx
committed
fixed typos in test description and comment

stockingstocker
authored and
hmhngx
committed
- Added group data imports and state
hmhngx
committed
add translation keys for defaultgraphicunit
hmhngx
committed
feat: enhance unit deletion error handling with improved UX
hmhngx
committed
feat: add translation keys for enhanced unit deletion error messages
hmhngx
committed
feat: add server-side transaction handling for unit deletion
hmhngx
committed
refactor: simplify unit deletion error handling
hmhngx
committed
refactor: remove obsolete unit deletion translation keys
hmhngx
committed
refactor: simplify unit deletion route
hmhngx
committed
refactor: make actionConfirmMessage optional in ConfirmActionModalComponent
hmhngx
committed
refactor: improve unit deletion error message formatting and logic
hmhngx
committed
feat: add translation key for improved meter deletion error message
hmhngx
committed
remove confirm since only does cancel

huss
authored and
hmhngx
committed
Added suffix unit deletion to unit deletion route to keep consistent with edit.

esummerh
authored and
hmhngx
committed
Added deletion logic to delete related conversions and units when a conversion involving a suffix unit is deleted.

esummerh
authored and
hmhngx
committed
Added warning message to user interface when the user tries to delete a conversion involving a suffix unit.

esummerh
authored and
hmhngx
committed
feat: add suffix unit dependency checking and fix async cleanup
hmhngx
committed
feat: add DB layer methods for cascading suffix deletions
hmhngx
committed
feat: enhance conversion/unit deletion with dependency validation
hmhngx
committed
feat: enhance UI warnings for suffix unit deletions
hmhngx
committed
fix: correct import path and remove duplicate property
hmhngx
committed

Enhance suffix unit conversion deletion with dependency checks#1561hmhngx wants to merge 319 commits intoOpenEnergyDashboard:developmentOpenEnergyDashboard/OED:developmentfrom hmhngx:deletion-updatehmhngx/OED:deletion-updateCopy head branch name to clipboard

Commits

Commits on Dec 11, 2025

Enhance suffix unit conversion deletion with dependency checks#1561
hmhngx wants to merge 319 commits intoOpenEnergyDashboard:developmentfrom
hmhngx:deletion-update