Skip to content
This repository was archived by the owner on Nov 10, 2023. It is now read-only.

Conversation

@iamacook
Copy link
Contributor

@iamacook iamacook commented Feb 8, 2022

What it solves

Resolves #3458

How this PR fixes it

The Safe-specific sidebar is only shown when a Safe address exists in the URL. (The 'loaded' Safe remains in the store so it is necessary to use the URL as a flag as well).

How to test it

  1. Open a Safe.
  2. Click the Safe logo.
  3. Observe that the Safe-specific sidebar is no longer visible.

Screenshots

image

@iamacook iamacook requested a review from katspaugh February 8, 2022 16:38
@github-actions
Copy link

github-actions bot commented Feb 8, 2022

CLA Assistant Lite All Contributors have signed the CLA.

@github-actions
Copy link

github-actions bot commented Feb 8, 2022

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type Occurrences Fixable
Errors 0 0
Warnings 0 0
Ignored 0 N/A
  • Result: ✅ success
  • Annotations: 0 total

Report generated by eslint-plus-action

@github-actions
Copy link

github-actions bot commented Feb 8, 2022

Deployment links

🟠 Rinkeby Mainnet 🟣 Polygon 🟡 BSC Arbitrum 🟢 Gnosis Chain

@coveralls
Copy link

Pull Request Test Coverage Report for Build 1813497675

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.008%) to 33.495%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/components/AppLayout/Sidebar/SafeHeader/index.tsx 0 2 0.0%
Files with Coverage Reduction New Missed Lines %
src/components/AppLayout/Sidebar/SafeHeader/index.tsx 1 0%
Totals Coverage Status
Change from base Build 1813118881: -0.008%
Covered Lines: 3211
Relevant Lines: 8551

💛 - Coveralls

@github-actions
Copy link

github-actions bot commented Feb 8, 2022

E2E Tests Failed
Check the results here: https://github.com/gnosis/safe-react-e2e-tests/actions/runs/1813531328

Failed tests:

  • ❌ Read-only transaction creation and review Read-only transaction creation and review

@usame-algan usame-algan self-requested a review February 9, 2022 09:01
Copy link
Contributor

@usame-algan usame-algan left a comment

Choose a reason for hiding this comment

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

Nice fix!

@katspaugh
Copy link
Member

AFAIU the problem in the bug report isn't that the sidebar is open. It's the fact that the Safe address is displayed in it, although it failed to load.

@iamacook
Copy link
Contributor Author

iamacook commented Feb 9, 2022

AFAIU the problem in the bug report isn't that the sidebar is open. It's the fact that the Safe address is displayed in it, although it failed to load.

The sidebar still shows Safe-related information when you navigate away from the Safe 'view':

Clicking on the gnosis logo on the top left takes you to the welcome page, but the info of the safe you are currently in still shows

We were previously retrieving the Safe address from the URL, but now that it checks the store, it is ever present. It would previously not show Safe-related information in the sidebar

@francovenica
Copy link
Contributor

Looks good to me

@iamacook iamacook merged commit 831455f into dev Feb 9, 2022
@iamacook iamacook deleted the safe-sidebar branch February 9, 2022 11:45
@github-actions github-actions bot locked and limited conversation to collaborators Feb 9, 2022
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.

Welcome page with the safe still on the left sidebar

6 participants