#8 Add UI needed for restore bubble process

Sapludināts
jonathan sapludināja 40 revīzijas no kris/add_restore_ui uz master pirms 4 gadiem
kris komentēja pirms 4 gadiem
Vēl nav satura.
kris nomainīts nosaukums no WIP: (testing) Add UI needed for restore bubble process uz WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process pirms 4 gadiem
jonathan pieprasīja izmaiņas pirms 4 gadiem
jonathan atstāja komentāru

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

jonathan komentēja pirms 4 gadiem
Īpašnieks

please also resolve merge conflict. thanks.

please also resolve merge conflict. thanks.
kris nomainīts nosaukums no WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process uz Add UI needed for restore bubble process pirms 4 gadiem
jonathan komentēja pirms 4 gadiem
Īpašnieks

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

@kris can you please resolve merge conflicts? I changed some stuff with how stop/delete bubble works.
jonathan pieprasīja izmaiņas pirms 4 gadiem
jonathan atstāja komentāru

Looks great. Just a couple of questions/comments.

src/_store/system.module.js
@@ -180,0 +183,4 @@
title: messages.label_menu_network,
icon: messages.label_menu_networks_icon,
index: 1
});
jonathan komentēja pirms 4 gadiem

I think this should only be added if isAdmin is true, right?

I think this should only be added if `isAdmin` is true, right?
kris komentēja pirms 4 gadiem

Yeap, I’ll add that check.

Yeap, I'll add that check.
@@ -0,0 +34,4 @@
<div v-if="submitted && errors.has('restoreLongNetworkKey')" class="invalid-feedback d-block">
{{ errors.first('restoreLongNetworkKey') }}
</div>
</div>
jonathan komentēja pirms 4 gadiem

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.

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 komentēja pirms 4 gadiem

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.

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 komentēja pirms 4 gadiem

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

Sure, we can do the file upload in another PR.
jonathan pieminēja šo problēmu revīzijā pirms 4 gadiem
Add UI needed for restore bubble process (#8) Remove not used property from parameter object Show bubble menu item only to admins on nodes Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/account/NetworkPage.vue Reference method properly within this Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/account/NetworkPage.vue Show link properly for network in restoring state Load backups only when needed for netwrok in restoring state Link for restoring node Use latest stats for network Merge branch 'master' into kris/add_restore_ui Fix loading image for backups label Call backups API only once while starting network Show backup info in proper places only Show latest backup and allow queuing new one Use proper name for waiting restoring flag Merge branch 'master' into kris/add_restore_ui Add support for stopping metworks in restoring state Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/_store/system.module.js # src/account/NetworkPage.vue Use dinamyc labels-messages Report restore is started on the page Redirect to restore page in restoring mode Fix indent Split page for restore from login page Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/_store/system.module.js # src/account/NetworksPage.vue # src/auth/LoginPage.vue Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/auth/LoginPage.vue Merge branch 'master' into kris/add_restore_ui # Conflicts: # src/_services/user.service.js Reference configs properly through this object Add support for restoring bubble on running instance Show restore short key to user Enhance Danger Zone on UI Add restore network UI elements Redirect to single bubble page on non-sage networks Remove setup help message on the node itself Add bubble info page on end bubble server's WebUI Co-authored-by: Kristijan Mitrovic <kmitrovic@itekako.com> Reviewed-on: https://git.bubblev.org/bubblev/bubble-web/pulls/8
jonathan aizvēra šo izmaiņu pieprasījumu pirms 4 gadiem
jonathan izdzēsa atzaru kris/add_restore_ui pirms 4 gadiem

Recenzenti

jonathan pieprasīja izmaiņas pirms 4 gadiem
Izmaiņu pieprasījums tika sapludināts ar revīziju 142ac2090a.
Pierakstieties, lai pievienotos šai sarunai.
Nav recenzentu
Nav etiķešu
Nav atskaites punktu
Nav atbildīgo
2 dalībnieki
Paziņojumi
Izpildes termiņš

Izpildes termiņš nav uzstādīts.

Atkarības

Šim izmaiņu pieprasījumam pagaidām nav nevienas atkarības.

Notiek ielāde…
Vēl nav satura.