8000 Use `formattedPrice` for prices · Issue #597 · mas-cli/mas · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Use formattedPrice for prices #597

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintai 8000 ners and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
rgoldberg opened this issue Oct 25, 2024 · 0 comments · Fixed by #601
Closed

Use formattedPrice for prices #597

rgoldberg opened this issue Oct 25, 2024 · 0 comments · Fixed by #601
Labels
🐛 bug 🖨 output

Comments

@rgoldberg
Copy link
Contributor
rgoldberg commented Oct 25, 2024 •
edited
Loading

mas info & mas search --price currently output all prices, regardless of the currency, as $ <price in some currency, might not be USD>.

Use formattedPrice from iTunes Search results to print prices with the correct currency symbol & format.

Also, require fileSizeBytes & price for SearchResult.
rgoldberg added a commit to rgoldberg/mas that referenced this issue Oct 26, 2024
@rgoldberg
Use formattedPrice instead of price.
1b4c97f 
Improve price output.

Partial mas-cli#597

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
rgoldberg added a commit to rgoldberg/mas that referenced this issue Oct 26, 2024
@rgoldberg
Require fileSizeBytes & price for SearchResult.
fd051ea 
Resolve mas-cli#597

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
@rgoldberg rgoldberg mentioned this issue Oct 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
🐛 bug 🖨 output
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use formattedPrice for prices rgoldberg/mas
1 participant
@rgoldberg
0