Skip to content
This repository has been archived by the owner on Aug 22, 2022. It is now read-only.

[BB-3869] Update Hero page #762

Merged
merged 1 commit into from
Apr 16, 2021
Merged

[BB-3869] Update Hero page #762

merged 1 commit into from
Apr 16, 2021

Conversation

arjunsinghy96
Copy link
Contributor

@arjunsinghy96 arjunsinghy96 commented Apr 7, 2021

Updates hero page to conform to the new UI

Related Tickets BB-3869 Gitlab#721

Testing Instructions

  1. Checkout to this branch
  2. Visit http://localhost:3000/console/theming/hero
  3. Verify that the UI looks correct and matches this mock
  4. Verify that all the components from old Hero page are moved to sidebar.
  5. Verify that updating values in sidebar gets reflected in BetaTestApplication

Screenshots
Hero Edit screen
Screenshot from 2021-04-09 18-53-33

@arjunsinghy96 arjunsinghy96 changed the title WIP: BB-3869 Update Hero page [BB-3869] Update Hero page Apr 9, 2021
Copy link
Member

@farhaanbukhsh farhaanbukhsh left a comment

Choose a reason for hiding this comment

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

👍

  • I tested this: Checked on local devstack, this looks alright and changes are reflected
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

Copy link
Member

@farhaanbukhsh farhaanbukhsh left a comment

Choose a reason for hiding this comment

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

I have added few comments

frontend/src/console/components/Hero/Hero.tsx Outdated Show resolved Hide resolved
frontend/src/console/components/Hero/Hero.tsx Outdated Show resolved Hide resolved
Copy link
Member

@farhaanbukhsh farhaanbukhsh left a comment

Choose a reason for hiding this comment

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

👍

  • I tested this: I tested these changes with local setup
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation
  • I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

Updates hero page to conform to the new UI
@arjunsinghy96 arjunsinghy96 merged commit 0f1e905 into master Apr 16, 2021
@Agrendalath Agrendalath deleted the arjun/bb-3869-hero-page branch May 3, 2021 12:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants