Skip to content

Commit

Permalink
remove logic that removed items from the "to_install" and "to_remove"…
Browse files Browse the repository at this point in the history
… install skill settings. This will be done on the backend by the API that talks to the device.
  • Loading branch information
chrisveilleux committed Nov 29, 2018
1 parent f67f606 commit b9d690c
Showing 1 changed file with 0 additions and 36 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ export class InstallService {
case ('installing'): {
if (newSkillStatus === 'installed') {
this.statusNotifications.next([skillName, newSkillStatus]);
this.removeFromInstallQueue(skillName).subscribe();
} else if (newSkillStatus === 'failed') {
this.statusNotifications.next([skillName, 'install failed']);
} else {
Expand All @@ -91,7 +90,6 @@ export class InstallService {
case ('uninstalling'): {
if (!newSkillStatus) {
this.statusNotifications.next([skillName, 'uninstalled']);
this.removeFromUninstallQueue(skillName).subscribe();
} else if (newSkillStatus === 'failed') {
this.statusNotifications.next([skillName, 'uninstall failed']);
} else {
Expand Down Expand Up @@ -161,7 +159,6 @@ export class InstallService {
return this.http.put<Object>(
installerSettingsUrl,
{
action: 'add',
section: 'to_install',
skill_name: skillName
}
Expand All @@ -177,39 +174,6 @@ export class InstallService {
return this.http.put<Object>(
installerSettingsUrl,
{
action: 'add',
section: 'to_remove',
skill_name: skillName
}
);
}

/**
* Call the API to remove a skill to the Installer skill's 'to_install' setting.
*
* @param skillName: the skill being installed
*/
removeFromInstallQueue(skillName: string): Observable<Object> {
return this.http.put<Object>(
installerSettingsUrl,
{
action: 'remove',
section: 'to_install',
skill_name: skillName
}
);
}

/**
* Call the API to remove a skill to the Installer skill's 'to_remove' setting.
*
* @param skillName: the skill being removed
*/
removeFromUninstallQueue(skillName: string): Observable<Object> {
return this.http.put<Object>(
installerSettingsUrl,
{
action: 'remove',
section: 'to_remove',
skill_name: skillName
}
Expand Down

0 comments on commit b9d690c

Please sign in to comment.