Feat(Orgs): Add safes manually for orgs

[usame-algan]

What it solves

Resolves #5048

How this PR fixes it

• Adds a form to manually add a safe account in orgs

How to test it

1. Open an org
2. Add safe accounts
3. Press Add manually
4. Type in a safe address and chain id
5. Submit the form
6. Observe the safe appears in the list of safes and is checked
7. Observe the number in the Submit button updating

Screenshots

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://manualaddsafe--walletweb.review.5afe.dev/home?safe=eth:0xA77DE01e157f9f57C7c4A326eeE9C4874D0598b6

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

[usame-algan]

@jmealy I have issues getting the revalidation on the address field working when the user changes the chainId. Lets add a fix in case you have an idea otherwise I will create a bug ticket to handle it later.

[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.07 MB (🟡 +79.48 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!

New Pages Added

The following pages were added to the bundle from the code in this PR:

Page	Size (compressed)	First Load
/organizations/[orgId]	12.41 KB	1.08 MB
/organizations/[orgId]/members	11.42 KB	1.08 MB
/organizations/[orgId]/safeAccounts	5.45 KB	1.07 MB
/organizations/[orgId]/settings	2.64 KB	1.07 MB
/welcome/organizations	4.66 KB	1.07 MB

Thirty-two Pages Changed Size

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

Page	Size (compressed)	First Load
/	509 B (🟢 -1 B)	1.07 MB
/address-book	23.21 KB (🟡 +140 B)	1.09 MB
/apps	35.75 KB (🟡 +2.03 KB)	1.1 MB
/apps/custom	33.82 KB (🟡 +2.03 KB)	1.1 MB
/apps/open	55.36 KB (🟡 +1.75 KB)	1.12 MB
/balances	29.68 KB (🟢 -23 B)	1.1 MB
/balances/nfts	9.32 KB (🟢 -234 B)	1.08 MB
/bridge	2.55 KB (🟢 -4 B)	1.07 MB
/cookie	8.77 KB (🟡 +1 B)	1.08 MB
/home	66.57 KB (🟡 +7.31 KB)	1.13 MB
/licenses	2.46 KB (🟢 -1 B)	1.07 MB
/new-safe/advanced-create	26.26 KB (🟢 -201 B)	1.09 MB
/new-safe/create	25.4 KB (🟢 -198 B)	1.09 MB
/privacy	14.57 KB (🟡 +2 B)	1.08 MB
/settings	476 B (🟡 +1 B)	1.07 MB
/settings/data	1.79 KB (🟢 -1 B)	1.07 MB
/settings/modules	2.88 KB (🟢 -1.18 KB)	1.07 MB
/settings/notifications	7.49 KB (🟢 -13.82 KB)	1.07 MB
/settings/safe-apps	20.15 KB (🟡 +1.87 KB)	1.09 MB
/settings/security	2.34 KB (🟡 +2 B)	1.07 MB
/settings/setup	30.97 KB (🟡 +247 B)	1.1 MB
/share/safe-app	7.55 KB (🟢 -9 B)	1.07 MB
/stake	617 B (🟢 -2 B)	1.07 MB
/swap	763 B (🟡 +3 B)	1.07 MB
/terms	12.93 KB (🟡 +1 B)	1.08 MB
/transactions	99.46 KB (🟡 +2.88 KB)	1.16 MB
/transactions/history	99.42 KB (🟡 +2.88 KB)	1.16 MB
/transactions/messages	60.25 KB (🟡 +1.95 KB)	1.13 MB
/transactions/msg	56.5 KB (🟡 +1.95 KB)	1.12 MB
/transactions/queue	49.36 KB (🟡 +1.96 KB)	1.11 MB
/transactions/tx	48.72 KB (🟡 +1.96 KB)	1.11 MB
/welcome	6.93 KB (🟢 -2 B)	1.07 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	77.27%	14233/18421
🔴	Branches	56.1% (+0.01% 🔼)	3571/6365
🟡	Functions	62.13%	2123/3417
🟡	Lines	78.75%	12852/16321

Test suite run success

1838 tests passing in 251 suites.

Report generated by 🧪jest coverage report action from 35797ae

[jmealy]

You can use chains.eth, using config/chains.ts

[jmealy]

There is one bug where you can add the same safe multiple times to the list.

Also a couple of points, more for the whole flow rather than this modal specifically (don't need to handle these in this PR):

• when you have no safes selected, the submit button should be disabled
• For modals (both the full screen one and the add safe manually one) I think we usually have an X button in the top right to close it

Also what do you think about making it an AddressBookInput rather than AddressInput? So you can have safes suggested to you?

[jmealy]

@jmealy I have issues getting the revalidation on the address field working when the user changes the chainId. Lets add a fix in case you have an idea otherwise I will create a bug ticket to handle it later.

Ah I missed this comment. I noticed this while testing also, but it's also there in the add safe flow so it's an existing bug. Made a ticket for it here: #5067

[usame-algan]

Also what do you think about making it an AddressBookInput rather than AddressInput? So you can have safes suggested to you?

I tried this but had trouble getting the layout right because we switch between a read-only view and an input view. I suggest we create a ticket and give it a try if there is more time.