LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues596 Pull requests6 Projects Releases238 Packages Wiki Activity

Pass app version info to internal APIs #1583

Merged
daovist merged 3 commits from issue/1579 into master 2018-06-13 22:40:28 +02:00
Conversation 0 Commits 3 Files changed 1 +10 -13
1 changed files with 10 additions and 13 deletions
Showing only changes of commit 20dc820e06 - Show all commits

23
src/renderer/redux/actions/user.js
View file

@ -37,21 +37,18 @@ export function doFetchInviteStatus() {
}
export function doInstallNew() {
return dispatch => {
const payload = {app_version: pjson.version};
Lbry.status().then(status => {
payload.app_id = status.installation_id;
Lbry.version().then(version => {
payload.daemon_version = version.lbrynet_version;
payload.operating_system = version.os_system;
payload.platform = version.platform;
Lbryio.call('install', 'new', payload);
});
const payload = { app_version: pjson.version };
Lbry.status().then(status => {
payload.app_id = status.installation_id;
Lbry.version().then(version => {
payload.daemon_version = version.lbrynet_version;
payload.operating_system = version.os_system;
payload.platform = version.platform;
Lbryio.call('install', 'new', payload);
});
};
});
}
export function doAuthenticate() {
return dispatch => {
dispatch({
@ -66,7 +63,7 @@ export function doAuthenticate() {
});
dispatch(doRewardList());
dispatch(doFetchInviteStatus());
neb-b commented 2018-06-13 18:31:08 +02:00 (Migrated from github.com)
Review

Since we aren't storing this in the store, we don't need to wrap this call in a dispatch. You can just call doInstallNew() after.

Since we aren't storing this in the store, we don't need to wrap this call in a `dispatch`. You can just call `doInstallNew()` after.
dispatch(doInstallNew());
doInstallNew();
})
.catch(error => {
dispatch(doNotify({ id: MODALS.AUTHENTICATION_FAILURE }));