bubblev
/
bubble-web
8
0
Vork 0
Code Kwesties 0 Pull-aanvragen 0 Publicaties 0 Wiki Activiteit
Labels Mijlpalen
Nieuwe Pull aanvraag

#12 Fix restore UI

Samengevoegd
jonathan heeft 7 commits samengevoegd van kris/fix_restore_ui naar master 4 jaren geleden
Discussie 0 Commits 7 Bestanden gewijzigd 5
kris commented 4 jaren geleden

@jonathan please review

@jonathan please review
jonathan was toegekend door kris 4 jaren geleden
jonathan aangevraagde wijzigingen 4 jaren geleden
jonathan heeft een reactie achtergelaten

Just one question. Otherwise looks good.

src/_store/account.module.js
@@ -123,1 +124,3 @@
error => commit('restoreFailure', error)
error => {
commit('restoreFailure', error);
systemConfigs.restoringInProgress = false;
jonathan commented 4 jaren geleden

shouldn’t we move the systemConfigs.restoringInProgress = false; statements into the restoreSuccess and restoreFailure mutators?

shouldn't we move the `systemConfigs.restoringInProgress = false;` statements into the `restoreSuccess` and `restoreFailure` mutators?
jonathan heeft deze pull request gesloten 4 jaren geleden
jonathan heeft 4 jaren geleden de branch kris/fix_restore_ui verwijderd.

Reviewers

jonathan aangevraagde wijzigingen 4 jaren geleden
The pull request has been merged as 108d44cb49.
Log in om deel te nemen aan deze discussie.
Reviewers
Beoordeling aanvragen
Geen beoordelaars
 jonathan
Labels
Labels toepassen
Verwijder labels
Geen items
Geen label
Mijlpaal
Stel mijlpaal in
Verwijder mijlpaal
Geen items
Geen mijlpaal
Toegewezen aan
Gebruikers toewijzen
Verwijder toegewezen aan
Niet toegewezen
 jonathan
2 deelnemers
Notificaties
Vervaldatum

Geen vervaldatum ingesteld.

Afhankelijkheden

Deze pull-aanvraag heeft momenteel geen afhankelijkheden.

Schrijf Voorbeeld
Laden…
Annuleren
Opslaan
Er is nog geen inhoud.
Verwijder branch 'kris/fix_restore_ui'

Het verwijderen van een branch is permanent. Het KAN NIET ongedaan gemaakt worden. Wil je toch doorgaan?

Nee
Ja