Skip to content

Conversation

@dantovska
Copy link
Collaborator

What

Applied different styling to polish all forms relating sentinel:

  • Add sentinel db form - increase text, align footer buttons
  • Edit sentinel database - fix some title size, add spacings
  • Clone sentinel database - also fix title sizes, spacings, required forms

Note: The database information in the edit form is handled in this PR - #5149

Testing

Need to have sentinel database added.

  1. Add db form -> Redis sentinel button in the connectivity section - opens the targeted modal
Before After
image image
image image
  1. Edit db form -> open sentinel db for edit
Before After
image image
image image
  1. Clone sentile db -> in the edit form, Clone connection buttom in the top right
Before After
image image
image image

@github-actions
Copy link
Contributor

github-actions bot commented Nov 6, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.53% 20483/24819
🟡 Branches 67.74% 8775/12953
🟡 Functions 77.07% 5489/7122
🟢 Lines 82.94% 20055/24180

Test suite run success

5255 tests passing in 687 suites.

Report generated by 🧪jest coverage report action from f5637f1

@ArtemHoruzhenko
Copy link
Collaborator

@dantovska I feel like you have different text sizes for this container
Screenshot 2025-11-06 at 19 35 32

ArtemHoruzhenko
ArtemHoruzhenko previously approved these changes Nov 7, 2025
Copy link
Collaborator

@ArtemHoruzhenko ArtemHoruzhenko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

valkirilov
valkirilov previously approved these changes Nov 7, 2025
KrumTy
KrumTy previously approved these changes Nov 7, 2025
Copy link
Collaborator

@KrumTy KrumTy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@dantovska dantovska dismissed stale reviews from KrumTy, valkirilov, and ArtemHoruzhenko via f5637f1 November 7, 2025 07:59
@dantovska dantovska merged commit 38e6570 into main Nov 7, 2025
28 of 29 checks passed
@dantovska dantovska deleted the fe/feature/RI-7681-polish-sentinel branch November 7, 2025 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants