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

Error "required" is displayed for the safe name when Add safe is opened #2539

Closed
liliya-soroka opened this issue Jul 14, 2021 · 1 comment · Fixed by #2574
Closed

Error "required" is displayed for the safe name when Add safe is opened #2539

liliya-soroka opened this issue Jul 14, 2021 · 1 comment · Fixed by #2574
Assignees
@juampibermani
Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.
Milestone
3.11.0

Comments

@liliya-soroka
Copy link
Member

Description

Error "required" is displayed for the safe name when Add safe is opened

Environment

Steps to reproduce

  1. Go to https://pr2535--safereact.review.gnosisdev.com/mainnet/app/#
  2. connect wallet and click load existed safe
    Current result: Required error is displayed for the safe name field by default

Expected result

No error in the safe name field by default

Screenshots

image
@liliya-soroka liliya-soroka added the Bug 🐛 Something isn't working label Jul 14, 2021
@liliya-soroka liliya-soroka mentioned this issue Jul 14, 2021
Merged
@tschubotz tschubotz added the Minor Needs to be fixed within the next 1-3 public releases. label Jul 14, 2021
@katspaugh
Copy link
Member

Probably introduced in #2495.
Possibly this particular line (a new key prop).

@juampibermani juampibermani self-assigned this Jul 22, 2021
@juampibermani juampibermani mentioned this issue Jul 22, 2021
Merged
@dasanra dasanra added this to the 3.11.0 milestone Jul 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@juampibermani juampibermani

Labels
Bug 🐛 Something isn't working Minor Needs to be fixed within the next 1-3 public releases.
Projects
None yet
Milestone
3.11.0
Development

Successfully merging a pull request may close this issue.

Don't show required error when loading a safe 5afe/safe-react
5 participants
@liliya-soroka @katspaugh @tschubotz @dasanra @juampibermani