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

Add number formatter to Nameplace Capacity #1260

Merged

Conversation

dhaselhan
Copy link
Collaborator

No description provided.

Copy link

github-actions bot commented Nov 19, 2024

Backend Test Results

333 tests  ±0   128 ✅ ±0   45s ⏱️ -1s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0   205 🔥 ±0 

For more details on these errors, see this check.

Results for commit 40413e0. ± Comparison against base commit a11dcc9.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Nov 19, 2024

Frontend Test Results

  1 files  ±0  114 suites  ±0   36s ⏱️ ±0s
367 tests ±0  347 ✅ ±0  20 💤 ±0  0 ❌ ±0 
369 runs  ±0  349 ✅ ±0  20 💤 ±0  0 ❌ ±0 

Results for commit 40413e0. ± Comparison against base commit a11dcc9.

♻️ This comment has been updated with latest results.

@prv-proton prv-proton linked an issue Nov 19, 2024 that may be closed by this pull request
4 tasks
@prv-proton prv-proton self-requested a review November 19, 2024 20:34
prv-proton
prv-proton approved these changes Nov 19, 2024
Copy link
Collaborator

@prv-proton prv-proton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @dhaselhan , thanks for the implementation, Please find my observations below.

  • commas won't get added while editing, instead it gets formatted after editing is complete.
image image
  • Using NumberEditor component should fix this issue. Also if possible please fix the alignment to be same while editing and not editing.

@dhaselhan
Copy link
Collaborator Author

Hi @dhaselhan , thanks for the implementation, Please find my observations below.

  • commas won't get added while editing, instead it gets formatted after editing is complete.

image image

  • Using NumberEditor component should fix this issue. Also if possible please fix the alignment to be same while editing and not editing.

Good catch Prashanth! Updated the code to use the NumberEditor and it works great. I did look at other places for column alignment and it looks like Fuel Quantity follows the same left/right behavior and I couldn't find any cells that work the way you are suggesting.

# Conflicts:
#	frontend/src/views/FuelCodes/_schema.jsx
@dhaselhan dhaselhan force-pushed the feature/daniel-add-nameplace-capacity-formatting-1237 branch from d2d68bb to 40413e0 Compare November 19, 2024 22:49
@prv-proton
Copy link
Collaborator

prv-proton commented Nov 20, 2024

Hi @dhaselhan , thanks for the implementation, Please find my observations below.

  • commas won't get added while editing, instead it gets formatted after editing is complete.

image image

  • Using NumberEditor component should fix this issue. Also if possible please fix the alignment to be same while editing and not editing.

Good catch Prashanth! Updated the code to use the NumberEditor and it works great. I did look at other places for column alignment and it looks like Fuel Quantity follows the same left/right behavior and I couldn't find any cells that work the way you are suggesting.

Thanks for the changes Daniel, I had a ticket similar to this changes in FuelCode carbonIntensity, and Al suggested this change. you can refer here
frontend/src/views/FuelCodes/AddFuelCode/_schema.jsx
type: 'leftAligned'. We can confirm this with Al and Stuart tomorrow if these needs to be consistent across.

Tested and looks good.

@dhaselhan dhaselhan merged commit 2c0323d into release-0.2.0 Nov 20, 2024
7 of 9 checks passed
@dhaselhan dhaselhan deleted the feature/daniel-add-nameplace-capacity-formatting-1237 branch November 20, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LCFS - Add a comma to the values in the Nameplate capacity column of Fuel codes
2 participants