Het verwijderen van een branch is permanent. Het KAN NIET ongedaan gemaakt worden. Wil je toch doorgaan?
kris/download_account
naar master
4 jaren geleden
Het verwijderen van een branch is permanent. Het KAN NIET ongedaan gemaakt worden. Wil je toch doorgaan?
Note -- You might also need to make some changes to the
ActionPage.vue
to properly handle approvals for the account download.WIP: (not finished) Add support for downloading account datanaar Add support for downloading account data 4 jaren geledenNote: before updating this lib’s refence in the main bubble repo, please review and merge corresponding PR within that one ( bubblev/bubble#8 ) as there are some dependencies (English labels) there.
I realized I hadn’t submitted my previous review, sorry.
Overall this looks very good, just a few renaming changes and other minor things requested.
I would call this
requestAccountDownload
That was intentionally as some elements in code have
Request
as suffix, so I tried to make it different. Any other term to use here, or should I just change it torequest
?demand
?https://www.google.com/search?client=ubuntu&channel=fs&q=sysnonym+request&ie=utf-8&oe=utf-8
Yeah just changed to
request
. It’s OK if the prefix and suffix are bothrequest
, they have different semantics.the convention is for the commit methods to be named
methodNameRequest
,methodNameSuccess
andmethodNameFailure
, so these should berequestAccountDownloadRequest
,requestAccountDownloadSuccess
andrequestAccountDownloadFailure
Yes, that’s the one -
request...Request
. Do we need anotherhaters will hate
like comment here? :)Yeah just changed to
request
. It’s OK if the prefix and suffix are bothrequest
, they have different semantics.since this isn’t actually downloading the account data, but requesting an account data download, perhaps call this method
initiateAccountDownloadRequest
or something like thatThis one was already changed to
clickRequireAccountDownload
. Is that ok?@jonathan please check my answers here and confirm what to do...
Reviewers
8582fc2745
.