#8 Add UI needed for restore bubble process

Birleştirildi
jonathan 4 yıl önce kris/add_restore_ui içindeki 40 işleme master ile birleştirdi
kris 4 yıl önce yorum yaptı
Henüz bir içerik yok.
kris başlık WIP: (testing) Add UI needed for restore bubble process iken WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process olarak değiştirildi 4 yıl önce
jonathan 4 yıl önce değişiklik istedi
jonathan bir yorum bırak

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

jonathan 4 yıl önce yorum yaptı
Sahibi

please also resolve merge conflict. thanks.

please also resolve merge conflict. thanks.
kris başlık WIP: (testing & blocked by https://git.bubblev.org/bubblev/bubble/pulls/20 ) Add UI needed for restore bubble process iken Add UI needed for restore bubble process olarak değiştirildi 4 yıl önce
jonathan 4 yıl önce yorum yaptı
Sahibi

@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 4 yıl önce değişiklik istedi
jonathan bir yorum bırak

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 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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 4 yıl önce yorum yaptı

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

Sure, we can do the file upload in another PR.
jonathan 4 yıl önce işlemesinde bu konuyu işaret etti
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 4 yıl önce değişiklik isteğini kapattı
jonathan kris/add_restore_ui dalı silindi 4 yıl önce

Gözden Geçirenler

jonathan 4 yıl önce değişiklik istedi
Değişiklik isteği 142ac2090a olarak birleştirildi.
Bu konuşmaya katılmak için oturum aç.
Değerlendirici yok
Etiket Yok
Kilometre Taşı Yok
Atanan Kişi Yok
2 Katılımcı
Bildirimler
Bitiş Tarihi

Bitiş tarihi atanmadı.

Bağımlılıklar

Bu çekme isteği henüz bir bağımlılık içermiyor.

Yükleniyor…
Henüz bir içerik yok.