Comments (26)
I like the rephrasing, yes. The only thing is that from the first point it's unclear whether clicking the button would allow me to copy the owner policies myself or if that would be done automatically. So maybe something like "Click the button below to create a Safe. Your current owner and signature policies will be applied to it automatically."
from safe-react.
I understand. This modal would only show on the wallet detail page, as we can only migrate one wallet at a time.
@germartinez could you provide Giacomo with the information above?
from safe-react.
There is no endpoint needed for this feature.
We should create a link in the Multisig pointing to the Safe creation form in the Safe for Teams (gnosis-safe.io/open
) and passing the required data from the Multisig in a way that the Safe creation form in can be filled automatically.
The data required in the link is:
name
(Safe name)threshold
(Safe threshold)owneraddresses
(Array of the owner addresses)ownernames
(Array of the owners names in the same position as the owner addresses)
gnosis-safe.io/open?name=<name>&threshold=<threshold>&ownernames=<address1>,<address2>,<address3>&ownernames=<owner1>,<owner2>,<owner3>
from safe-react.
@giacomolicari This is also going to be relevant for you. Implementation is planned for late October / beginning of November, but feel free to already give feedback now or during the design stage.
from safe-react.
Sure @lukasschor , I can give feedback on this :)
from safe-react.
Popup:
Popup with checkbox checked:
"Migrate wallet" button leads here:
from safe-react.
Looks great. 👍
Another approach would be to already design the pop-up in the Safe for Teams design language to make a contrast to the current design language and hint towards the improved UX/UI. I'm definitely not saying this would be necessarily better, as it comes with its own drawbacks (e.g. it wouldn't look as "native" as it looks right now). But curious to hear your thoughts on this. @posthnikova
from safe-react.
I think it makes sense to have the Multisig popup look native, but maybe there can be a more prominent link with something like "Check out the new look!"? I think "Read more" sounds a bit tedious, it would be good to have something where the user can just get a quick look.
Also, is there a reason why "Dismiss" is so prominently highlighted as a button and "Migrate wallet" not at all? Is that just to be in line with the Multisig design? If not, I would change that.
Small typo in the Safe for Teams design: it should be "Make sure you have 0.00041..."
from safe-react.
@lukasschor @KristinaMayman I chose native style because it's less obtrusive and to reuse styles from Multisig design would be faster. If it doesn't work, we should show new styles more prominently.
I think "Read more" sounds a bit tedious, it would be good to have something where the
user can just get a quick look.
I just realised it goes to Safe website and there is not much more new information than in the popup, the same advantages are listed. There's no way to migrate wallet when you're on the Safe website so maybe we should not link there.
Also, is there a reason why "Dismiss" is so prominently highlighted as a button
This is the style used everywhere in Multisig. I would definitely show a more prominent "Migrate wallet" button
from safe-react.
UX sync:
- do a blog post
- make button more prominent, rename "Setup" or "Create Safe"
- Read more leads to blog post
- Mention that they have to move funds
from safe-react.
Changes after UX sync: button renamed to "Create new Safe", added a mention that they have to move funds to the new Safe. "Read more" leads to blog post. We need a blog post explaining how to migrate and a way to migrate from the Safe website.
from safe-react.
I would rephrase the last section to...
Migrating couldn't be easier:
- Click the button below to create a Safe with the exact same owner and signature policies as your existing Multisig.
- Try out the new interface and learn about the many benefits.
- As soon as you feel comfortable, start moving funds to your new Safe.
from safe-react.
@lukasschor @KristinaMayman Text changed.
from safe-react.
Looks good to me now.
@giacomolicari What do you think?
from safe-react.
I'm fine with it :) The only thing I would maybe change is that text Migration couldn't be easier: etc..
, I'd make it bigger to emphasize how the migration process work and which steps users have to take. What's your opinion? @lukasschor
from safe-react.
Agree, maybe we could make „Migration couldn‘t be easier“ as a title and also work with icons (1, 2, 3) and change the ones for the benefits to roman numbers or to symbols. Also, user experience has shown that gas efficiency is not really a big benefit for existing users, so I would leave it out.
from safe-react.
UX sync: Make "Migration couldn't be easier" bigger, remove 1-2-3 circles from top list, add 1-2-3 numbers to migration process, remove benefit 5
from safe-react.
Changes after UX sync @lukasschor @giacomolicari
from safe-react.
Looks good. Moving this to Dev Backlog. 👍
from safe-react.
@giacomolicari So from my side everything is fine and you could start implementing this feature. When do you think you will have time to work on this?
from safe-react.
Hi @lukasschor ,
I'd have a question about the flow. Do we show a migration modal for each wallet in the multisig app or just one for all wallets?
from safe-react.
I would show it every time any wallet is opened until the user ticks „don‘t show this again“. Even though that would mean that they would see this modal even after migrating a certain wallet. But if it annoys the user, he could always click that box.
The assumption is that as soon as they migrated one wallet, users would be able to just replicate others from the Safe interface. But in case that‘s not correct we might have to add an additional „Migrate to Safe“ button in the interface header or similar.
Do you think that makes sense?
from safe-react.
Yes I understand. My only concern is about data we have to send from the Multisig app to Safe's endpoint. I understand I have to collect some data from the wallet, like list of owners and wallet's address.
In case a user is on the index page, we would send the whole list of multisigs a user holds, like:
[
{
"name":"Multisig 1",
"address":"0x.....",
"owners":[
{
"name":"Owner 1",
"address":"0x...."
},
{
"name":"Owner 2",
"address":"0x...."
}
]
},
{
"name":"Multisig 2",
"address":"0x.....",
"owners":[
{
"name":"Owner 1",
"address":"0x...."
},
{
"name":"Owner 2",
"address":"0x...."
}
]
}
]
instead, if the user is on a wallet's detail page, we would send just data related to that wallet
[
{
"name":"Multisig 1",
"address":"0x.....",
"owners":[
{
"name":"Owner 1",
"address":"0x...."
},
{
"name":"Owner 2",
"address":"0x...."
}
]
}
]
Could you also provide me with the Safe's endpoint and what kind of http request to send? (GET, POST)
thank you @lukasschor
from safe-react.
@germartinez Because currently we do not allow users to create a Safe without an active wallet connection, we show this screen if the wallet gets disconnected in the creation flow. Does this cause any issues for this migration feature? Because the user would land on the interface and first have to connect a wallet but we should definitely not show this blank screen.
from safe-react.
Right now, that would be the behaviour. The URL would still have the data, so whenever the provider is connected, the form would be shown automatically with all the data.
from safe-react.
@giacomolicari For the public launch we will rename the Safe for Teams to "Gnosis Safe Multisig". So there are two changes needed in the text of the migration screen.
from safe-react.
Related Issues (20)
- highlighting paused state of the safe token contract in the asset list HOT 1
- Signing with Ledger on Avalanche not working
- Add more tests for the safe apps signing feature
- Safe Ledger support on Aurora HOT 3
- [Desktop app] Invalid JSON RPC response: "project id required in the url" HOT 3
- EIP-712 signing not working HOT 1
- Create a new safe when already having a safe HOT 3
- Ledger not connecting after update HOT 1
- Optimism: creating a Safe – gas limit < 21000 HOT 1
- Enable Optimism support for Tally Ho HOT 5
- Error: A "infuraKey" or a "rpc" object with a parameter of 4 must be included in the WalletConnect initialization object HOT 2
- Allow DELEGATECALL as a UI option
- Gnosis safe transaction stuck HOT 1
- txStatus Cancelled HOT 4
- Enable Arbitrum support for Tally Ho on Safe HOT 3
- custom deployed safe not found HOT 3
- missing builds since 3.22.1 HOT 3
- Allow empty string to be passed HOT 2
- Sepolia Testnet Version HOT 6
- Always indexing HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from safe-react.