Add UI needed for restore bubble process

Обговорення 2 Коміти 40 Змінені файли 10

Тут ще немає жодного змісту.

kris змінився заголовок з ~~WIP: (testing) Add UI needed for restore bubble process~~ на WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process 4 роки тому

jonathan зробив запит змін 4 роки тому

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

please also resolve merge conflict. thanks.

kris змінився заголовок з ~~WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process~~ на Add UI needed for restore bubble process 4 роки тому

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

jonathan зробив запит змін 4 роки тому

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 прокоментував(ла) 4 роки тому

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 прокоментував(ла) 4 роки тому

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 прокоментував(ла) 4 роки тому

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

jonathan згадано цю проблему в коміті 4 роки тому

jonathan closed this pull request 4 роки тому

jonathan видалена гілка kris/add_restore_ui 4 роки тому

Рецензенти

jonathan зробив запит змін 4 роки тому

Запит на злиття був влитиий як 142ac2090a.

Підпишіться щоб приєднатися до обговорення.

jonathan

Без мітки

Етап відсутній

Немає виконавеця

2 учасників

Сповіщення

Дата завершення

Термін виконання не встановлений.

Залежності

Цей запит на злиття в даний час не має залежностей.

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

Рецензенти