Add UI needed for restore bubble process

Diskussion 2 Commits 40 Geänderte Dateien 10

Hier gibt es bis jetzt noch keinen Inhalt.

kris hat den Titel von ~~WIP: (testing) Add UI needed for restore bubble process~~ zu WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process vor 4 Jahren geändert

jonathan hat vor 4 Jahren Änderungen angefragt

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

please also resolve merge conflict. thanks.

kris hat den Titel von ~~WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process~~ zu Add UI needed for restore bubble process vor 4 Jahren geändert

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

jonathan hat vor 4 Jahren Änderungen angefragt

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 hat vor 4 Jahren kommentiert

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 hat vor 4 Jahren kommentiert

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 hat vor 4 Jahren kommentiert

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

jonathan hat dieses Issue vor 4 Jahren aus einem Commit referenziert

jonathan hat diesen Pull-Request vor 4 Jahren geschlossen

jonathan löschte die Branch kris/add_restore_ui vor 4 Jahren

Reviewer

jonathan hat vor 4 Jahren Änderungen angefragt

Der Pull Request wurde als 142ac2090a zusammengeführt.

Anmelden, um an der Diskussion teilzunehmen.

jonathan

Kein Label

Kein Meilenstein

Niemand zuständig

2 Beteiligte

Nachrichten

Fällig am

Kein Fälligkeitsdatum gesetzt.

Abhängigkeiten

Dieser Pull-Request hat momentan keine Abhängigkeiten.

@@ -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

Reviewer