fix: Enable Add proposer button for owners only

[usame-algan]

What it solves

Resolves #4743

How this PR fixes it

• Adds a new HoC called OnlyOwner and uses it for the Add proposer button instead of the CheckWallet component

How to test it

1. Open a Safe
2. Go to the Settings page
3. Connect with an owner
4. The Add proposer button is enabled
5. Connect with a different wallet e.g. spending limit beneficiary, proposer, nested safe owner, recoverer
6. Observe that the add proposer button is disabled

Checklist

• I've tested the branch on mobile 📱
• I've documented how it affects the analytics (if at all) 📊
• I've written a unit/e2e test for it (if applicable) 🧑‍💻

[github-actions]

Branch preview

✅ Deploy successful!

Website:
https://only_owner--walletweb.review.5afe.dev/home?safe=eth:0xA77DE01e157f9f57C7c4A326eeE9C4874D0598b6

Storybook:
https://only_owner--walletweb.review.5afe.dev/storybook/

[usame-algan]

I decided to write a new component instead of adjusting CheckWallet because its already way too complex and I don't want to take the risk of breaking another place where CheckWallet is used.

[github-actions]

📦 Next.js Bundle Analysis for @safe-global/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

⚠️ Global Bundle Size Increased

Page	Size (compressed)
global	1.04 MB (🟡 +54.14 KB)

Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

Thirty-one Pages Changed Size

The following pages changed size from the code in this PR compared to its base branch:

Page	Size (compressed)	First Load
/	508 B (🟢 -2 B)	1.04 MB
/address-book	23.21 KB (🟡 +142 B)	1.06 MB
/apps	35.97 KB (🟡 +2.24 KB)	1.08 MB
/apps/custom	34.03 KB (🟡 +2.25 KB)	1.07 MB
/apps/open	55.57 KB (🟡 +1.96 KB)	1.1 MB
/balances	29.88 KB (🟡 +176 B)	1.07 MB
/balances/nfts	9.52 KB (🟢 -24 B)	1.05 MB
/bridge	2.56 KB (🟡 +2 B)	1.04 MB
/cookie	8.77 KB (🟡 +1 B)	1.05 MB
/home	61.54 KB (🟡 +2.28 KB)	1.1 MB
/new-safe/advanced-create	26.38 KB (🟢 -70 B)	1.07 MB
/new-safe/create	25.52 KB (🟢 -71 B)	1.07 MB
/privacy	14.57 KB (🟡 +2 B)	1.06 MB
/settings/appearance	2.25 KB (🟡 +2 B)	1.04 MB
/settings/environment-variables	3.27 KB (🟢 -1 B)	1.04 MB
/settings/modules	4.06 KB (🟡 +1 B)	1.05 MB
/settings/notifications	6.33 KB (🟢 -14.99 KB)	1.05 MB
/settings/safe-apps	20.35 KB (🟡 +2.08 KB)	1.06 MB
/settings/security	2.34 KB (🟡 +1 B)	1.04 MB
/settings/setup	30.97 KB (🟡 +241 B)	1.07 MB
/share/safe-app	7.56 KB (🟢 -5 B)	1.05 MB
/stake	618 B (🟢 -1 B)	1.04 MB
/swap	761 B (🟡 +1 B)	1.04 MB
/terms	12.93 KB (🟡 +1 B)	1.05 MB
/transactions	99.46 KB (🟡 +2.89 KB)	1.14 MB
/transactions/history	99.42 KB (🟡 +2.89 KB)	1.14 MB
/transactions/messages	60.25 KB (🟡 +1.95 KB)	1.1 MB
/transactions/msg	56.5 KB (🟡 +1.95 KB)	1.1 MB
/transactions/queue	49.36 KB (🟡 +1.96 KB)	1.09 MB
/transactions/tx	48.72 KB (🟡 +1.96 KB)	1.09 MB
/welcome/accounts	409 B (🟡 +2 B)	1.04 MB

Details

Only the gzipped size is provided here based on an expert tip.

First Load is the size of the global bundle plus the bundle for the individual page. If a user were to show up to your website and land on a given page, the first load size represents the amount of javascript that user would need to download. If next/link is used, subsequent page loads would only need to download that page's bundle (the number in the "Size" column), since the global bundle has already been downloaded.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

Next to the size is how much the size has increased or decreased compared with the base branch of this PR. If this percentage has increased by 20% or more, there will be a red status indicator applied, indicating that special attention should be given to this.

[github-actions]

Coverage report for apps/web

St.❔	Category	Percentage	Covered / Total
🟡	Statements	74.11% (-0.14% 🔻)	14578/19670
🔴	Branches	51.7% (-0.31% 🔻)	3478/6727
🔴	Functions	57.12% (-0.23% 🔻)	2065/3615
🟡	Lines	75.66% (-0.14% 🔻)	13224/17479

Show files with reduced coverage 🔻

St.❔	File	Statements	Branches	Functions	Lines
🟢	src/utils/chains.ts	93.1%	90% (-2.31% 🔻)	100%	100%
🟢	... / utils.ts	90.91% (-0.34% 🔻)	71.88% (-8.77% 🔻)	90% (-0.48% 🔻)	95.08% (-0.23% 🔻)
🟢	... / firebase.ts	86.96% (+7.79% 🔼)	72.73% (-4.2% 🔻)	100%	89.47% (+9.47% 🔼)
🟢	... / useNotificationPreferences.ts	86.36% (-2.65% 🔻)	40% (-26.67% 🔻)	80.65% (-1.17% 🔻)	85.54% (-2.69% 🔻)
🟢	... / SignForm.tsx	91.94% (+0.13% 🔼)	85.29% (-0.42% 🔻)	100%	93.1% (+0.12% 🔼)
🟡	... / index.tsx	68.52% (-0.57% 🔻)	10.26%	66.67%	69.23% (-0.58% 🔻)
🟡	... / index.tsx	67.65% (-0.92% 🔻)	42.11%	20%	69.84% (-0.93% 🔻)

Test suite run success

1758 tests passing in 238 suites.

Report generated by 🧪jest coverage report action from 25b42b4

[tmjssz]

Looks good to me.
Perhaps we could also use the new OnlyOwner component in other places that are connected to the proposer logic? e.g. UpsertProposer

[francovenica]

Isssue:
When I'm not connected it tells me that I have a wallet connecte that is not a signer:

The rest looks good

[usame-algan]

When I'm not connected it tells me that I have a wallet connecte that is not a signer:

Pushed a fix. It should now say to connect a wallet

[francovenica]

The fix works fine.

LGTM