Skip to content

Conversation

@DanielMoya-Intel
Copy link
Contributor

PR Checklist

  • [ N/A ] Unit Tests have been added for new changes
  • [ N/A ] API tests have been updated if applicable
  • [ N/A ] All commented code has been removed
  • [ N/A ] If you've added a dependency, you've ensured license is compatible with Apache 2.0 and clearly outlined the added dependency.

What are you changing?

Before the fix, when creating a new profile, if validation or other errors occurred, the informational messages at the top were truncated.

Fixes #2486

Anything the reviewer should know when reviewing this PR?

Tested manually by locally running the UI.

@DanielMoya-Intel
Copy link
Contributor Author

Here's is how to fix looks on the UI:

image

rsdmike
rsdmike previously approved these changes Mar 3, 2025
@rsdmike rsdmike enabled auto-merge (rebase) March 3, 2025 22:23
@rsdmike rsdmike changed the title fix #2486: truncated profile error messages fix: truncated profile error messages (#2486) Mar 3, 2025
@rsdmike rsdmike changed the title fix: truncated profile error messages (#2486) fix: truncated profile error messages Mar 3, 2025
@rsdmike
Copy link
Member

rsdmike commented Mar 3, 2025

@DanielMoya-Intel looks like just a npm run prettify to ensure the code is correct. I also updated the title to ensure it follows conventional commits.

Before the fix, when creating a new profile, if validation or other errors occurred, the informational messages at the top were truncated.

Fixes device-management-toolkit#2486

Signed-off-by: Moya Sanchez, Daniel <[email protected]>
Signed-off-by: Byron Marohn <[email protected]>
Signed-off-by: Baltodano Zamora, Jefry <[email protected]>
Signed-off-by: Vela, Mano <[email protected]>
auto-merge was automatically disabled March 4, 2025 00:44

Head branch was pushed to by a user without write access

@agustinfrancesa agustinfrancesa force-pushed the fix/truncated-profile-notices-2468 branch from 8c9f6ff to fa38b31 Compare March 4, 2025 00:44
@agustinfrancesa
Copy link
Contributor

Fix prettier validation errors, by commit amend

@rsdmike rsdmike merged commit e8c3c25 into device-management-toolkit:main Mar 4, 2025
14 checks passed
@RosieAMT
Copy link

RosieAMT commented Mar 4, 2025

🎉 This PR is included in version 3.32.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

User notices when creating a Profile are truncated

4 participants