Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

feat(filter): currencyFilter takes number of decimals #3461

Closed
wants to merge 1 commit into from
Closed

feat(filter): currencyFilter takes number of decimals #3461

wants to merge 1 commit into from

Conversation

doshprompt
Copy link
Contributor

currencyFilter accepts second argument as number of decimal places

feat(filter): currencyFilter takes number of decimals
4e8a4bd 
currencyFilter accepts second argument as number of decimal places
@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

@petebacondarwin
Copy link
Contributor

@doshprompt - Thanks for the PR - please can you ensure you have signed the CLA?

@btford
Copy link
Contributor

btford commented Aug 21, 2013

dupe of #3642

@btford btford closed this Aug 21, 2013
IgorMinar pushed a commit that referenced this pull request Oct 11, 2014
@doshprompt @IgorMinar
feat(currencyFilter): add fractionSize as optional parameter
20685ff 
currencyFilter accepts number of decimals to round off to

Closes #3642
Closes #3461
Closes #3642
Closes #7922
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@doshprompt @mary-poppins @petebacondarwin @btford