feedhenry / fh-fhc

FeedHenry CLI, the Command Line Interface to FeedHenry
Other
26 stars 66 forks source link

RHMAP-19611 - Adding missing command 'fhc admin appstore storeitems add --id=<id>' #437

Closed camilamacedo86 closed 6 years ago

camilamacedo86 commented 6 years ago

Motivation

The command to add a store item[1] related to the point is missing after apply the V3 standard, improve the outputs and add the unit tests into the PR [2]. This PR is in order to add the missing command which has the endpoint [3]

Local tests

camiladeomacedo@Camilas-MBP ~/redhat/fh-fhc (RHMAP-19611) $ ./bin/fhc.js  admin appstore storeitems add --id=3v4qjzyktlnfvevje5unoqgb
Item Store added successfully.
camiladeomacedo@Camilas-MBP ~/redhat/fh-fhc (RHMAP-19611) $ ./bin/fhc.js  admin appstore storeitems add --id=3v4qjzyktlnfvevje5unoqgb --json
{
  "status": "ok"
}
camiladeomacedo@Camilas-MBP ~/redhat/fh-fhc (RHMAP-19611) $ 

[1] - https://github.com/feedhenry/fh-fhc/commit/9ae4cc6f4f1007cf19a0829903ecd29e3bf9b5f9#diff-bb7c1835b9d57a46ca5f53d0201d08ebL84 [2] - https://github.com/feedhenry/fh-fhc/pull/378 [3] - https://access.redhat.com/documentation/en-us/red_hat_mobile_application_platform_hosted/3/html/platform_api/mobile-app-management-api#mobile-app-management-api-add-storeitem-to-app-store-admin

camilamacedo86 commented 6 years ago

@mmusil could you check this one?

camilamacedo86 commented 6 years ago

@davidffrench following the PR regards the JIRA open by @mmusil

camilamacedo86 commented 6 years ago

@mmusil could you recheck here? I fixed the unit test. It was wrong, really thank you for your review.