Add UI needed for restore bubble process

Conversación 2 Commits 40 Archivos modificados 10

Aún no existe contenido.

kris cambió el título de ~~WIP: (testing) Add UI needed for restore bubble process~~ a WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process hace 4 años

jonathan cambios solicitados hace 4 años

Looks good to me. Would have to try it to tell you more.

please also resolve merge conflict. thanks.

kris cambió el título de ~~WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process~~ a Add UI needed for restore bubble process hace 4 años

@kris can you please resolve merge conflicts? I changed some stuff with how stop/delete bubble works.

jonathan cambios solicitados hace 4 años

Looks great. Just a couple of questions/comments.

src/_store/system.module.js

src/auth/RestorePage.vue

												
			@@ -0,0 +34,4 @@
		
			                    <div v-if="submitted && errors.has('restoreLongNetworkKey')" class="invalid-feedback d-block">
		
			                        {{ errors.first('restoreLongNetworkKey') }}
		
			                    </div>
		
			                </div>

jonathan comentado hace 4 años

Can we do this as a file-upload instead? We should make also sure that the downloaded restore key has a .txt file extension in any case; please confirm.

kris comentado hace 4 años

Sure, but can we do this in another PR?
It would be also good to download this key as txt file at the first place in bubble’s page on the node itself (which is why we need that bubble page there - I see you removed that comment, so I guess you tried it).

I’ll be implementing this today (if merging master and retesting goes ok), but in separate branch starting from this one.

jonathan comentado hace 4 años

Sure, we can do the file upload in another PR.

jonathan referenció esta incidencia en un commit hace 4 años

jonathan cerró este pull request hace 4 años

jonathan rama eliminada kris/add_restore_ui hace 4 años

Revisores

jonathan cambios solicitados hace 4 años

El Pull Request se ha fusionado como 142ac2090a.

Inicie sesión para unirse a esta conversación.

jonathan

Sin etiquetas

Sin Milestone

No asignados

2 participantes

Notificaciones

Fecha de vencimiento

Sin fecha de vencimiento.

Dependencias

Este pull request actualmente no tiene ninguna dependencia.

@@ -180,0 +183,4 @@
                 title: messages.label_menu_network,
                 icon: messages.label_menu_networks_icon,
                 index: 1
             });

#8 Add UI needed for restore bubble process

Revisores