search

SimplyStaking / panic

PANIC Monitoring and Alerting For Blockchains
Apache License 2.0
82 stars 31 forks source link

UAT Batch Fixes: Group 1 #351

Closed zimaah closed 1 year ago

zimaah commented 1 year ago

User Story

Group 1 includes a set of findings from the UAT.

Description

  1. In the Channels step, the 'All' filter (in the channel types filter) appears at the bottom, forcing the user to scroll to it due to the insufficient modal height.
  2. Substrate Node Form height
  3. The word "base-chain" might need a review regarding the orthography... I think it should be "base chain".
  4. Base chain select disabled (For General base chain only)

Requirements

  1. The "All" option must be placed at the top of the channel types filter
  2. The --height CSS prop must be adjusted for when the nodes modal is opened for a Substrate base chain
  3. We need to perform a search & replace from base-chain to base chain
  4. When adding a General base chain, if we move to the Channels step, and then back to the Sub-chain step, the base chain svc-select becomes slightly lighter, giving the impression that it's disabled. We need to check its CSS.

Acceptance criteria

All items must be sorted following the requirements. If something isn't clear, please reach out to the team as soon as possible.

itsciccio commented 1 year ago

Additional bug to fix:

During the installer journey, when setting the base chain to 'General', advancing to the next step and returning to the previous step, the node op is faced with the Sub Chain Step. When swapping base chain and hence deleting the config, the 'General' option is not present even though a General base chain does not exist.

This fix will also be catered for in this ticket.