[HUM-170]: fix: error notification is displayed twice when should be once

[mpblocky]

Issue tracking

Closes: HUM-170

Context behind the change

• Added wallet connect initialization check
• Relaxed isReady condition as the mutation.data could be empty

How has this been tested?

• tested the expected behaviour on production build

Release plan

normal

Potential risks; What to monitor; Rollback plan

N/A

[vercel]

@mpblocky is attempting to deploy a commit to the HUMAN Protocol Team on Vercel.

A member of the Team first needs to authorize it.

[mpblocky]

@dnechay fix ready :)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
human-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Feb 21, 2025 1:04pm

[dnechay]

Fix lgtm, but wallet connect functionality is broken for operator now

[mpblocky]

@dnechay I added this notification as well, becuase there was just a misleading "cannot sign message" when user had invalid network selected in the web3 provider.

I'm looking into the operator problem, where exactly?

[dnechay]

I added this notification as well, becuase there was just a misleading "cannot sign message" when user had invalid network selected in the web3 provider.

@mpblocky great addition, thank you!

I'm looking into the operator problem, where exactly?

On the main page, regular "Operator Sign In" button doesn't work for the account I have on STG. Same for "Sign Up". I suppose other functionality might be affected as well (if any)

[mpblocky]

@dnechay thanks, I will investigate and check other places that could get affected

[mpblocky]

@dnechay reverting the changes in wallet connect context were sufficient, so no regression tests are needed as changes in worker profile were enough to fix the notification problem :)