search

nspcc-dev / neofs-dev-env

NeoFS local Development and Testing environment
https://fs.neo.org
GNU General Public License v3.0
9 stars 25 forks source link

`wallet/wallet.json` is useless #266

Open notimetoname opened 1 year ago

notimetoname commented 1 year ago

With https://github.com/nspcc-dev/neofs-dev-env/pull/238/commits/61fd192eaa410985e14d5466eb2a966c137cb545 that wallet becomes as useful as any other just-generated wallet. None of make commands deposits on it, no container can be created with it, etc. But on the other hand it is still mentioned in README (and in doc dir too). I would add a separate make target that replenishes wallet/wallet.json and add it as a prerequisite for any up command. @roman-khimov

roman-khimov commented 1 year ago

IIUC documentation mentions that a GAS transfer should be made, if you can make it easier to use -- OK.

notimetoname commented 1 year ago

IIUC documentation mentions that a GAS transfer should be made

@roman-khimov, updated the link(s). I read it as "do make prepare.ir and wallet/wallet.json is your door to the NeoFS". And that is how I used to use it. Now it is just an empty wallet. Making a transfer to it means a user knows what neo-go is and how to use it. So I think we should either drop it or make it great (useful) again. BTW, why neo-go needs a neofs balance? I think smth is wrong here.

cthulhu-rider commented 8 months ago

being a DevEnv user, i'd appreciate having both fixed "zero" wallet and the "one with some deposit sufficient for ~80% needs" from the box, e.g.: