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

v0.27.3 #899

Merged
merged 13 commits into from
Dec 7, 2024
Merged

v0.27.3 #899

merged 13 commits into from
Dec 7, 2024

Conversation

thisismayuresh
Copy link
Collaborator

No description provided.

Mayur and others added 13 commits November 26, 2024 19:22
**Merging this branch will resolve the security warning related to**
#881
)

**Fix crash due to undefined min/max values causing toFixed error**

When editing a column with only one defined min or max value, the app
would crash with the error "cannot read properties of undefined (reading
'toFixed')".
To resolve this, we updated the validation logic to allow undefined
values for min and max. This prevents the app from attempting to format
undefined values, eliminating the crash and allowing users to safely
edit columns with incomplete min/max values.
)

**Fix crash due to undefined min/max values causing toFixed error**

When editing a column with only one defined min or max value, the app
would crash with the error "cannot read properties of undefined (reading
'toFixed')".
To resolve this, we updated the validation logic to allow undefined
values for min and max. This prevents the app from attempting to format
undefined values, eliminating the crash and allowing users to safely
edit columns with incomplete min/max values.
Copy link

nx-cloud bot commented Dec 7, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 9920212. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

libs/dal/src/repositories/base-repository.ts Dismissed Show dismissed Hide dismissed
@chavda-bhavik chavda-bhavik merged commit c3b8734 into main Dec 7, 2024
7 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants