amplication
amplication copied to clipboard
fix(client): fix message broker settings page
Close: #7371
PR Details
- This pull request solves issue #7371 where the
message-broker
Resource type should not have all the settings that aservice
resource has - This PR checks the current resource type and displays the settings as per the type of the Resource.
PR Checklist
- [x] Tests for the changes have been added
- [x]
npm test
doesn't throw any error
Screenshots
With service
resource:
With message-broker
resource:
Hey @abrl91, could you please take a look at this and let me know if any changes need to be made? I'm relatively new to open source and would appreciate any pointers or advice :)
☁️ Nx Cloud Report
CI is running/has finished running commands for commit 50df4f3cdd460eba08bd60313d65851fe8fb7c6b. 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 4 targets
Sent with 💌 from NxCloud.