amplication icon indicating copy to clipboard operation
amplication copied to clipboard

fix(client): fix message broker settings page

Open Jayanth-Parthsarathy opened this issue 9 months ago • 1 comments

Close: #7371

PR Details

  • This pull request solves issue #7371 where the message-broker Resource type should not have all the settings that a service 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:

image

With message-broker resource:

image

Jayanth-Parthsarathy avatar May 07 '24 03:05 Jayanth-Parthsarathy

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 :)

Jayanth-Parthsarathy avatar May 07 '24 03:05 Jayanth-Parthsarathy

☁️ 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.

nx-cloud[bot] avatar May 13 '24 11:05 nx-cloud[bot]