Skip to content
New issue

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

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

0.3.0 #27

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

0.3.0 #27

wants to merge 7 commits into from

Conversation

ducdigital
Copy link

This version is maintained here: https://github.com/ducdigital/money.js
Feel free to give comment, add more tests and new feature in a PR.

In this version all API of money.js remain the same but introduce new features to this project.

Change Log

0.3.0

@PierrickP
Copy link

👍

@asfktz
Copy link

asfktz commented Feb 12, 2018

Hi @ducdigital!
This PR looks so promising.
Multiple instance support is definitely needed.

What stops it from being merged?
Is there anything I can do to help?

@jsw
Copy link

jsw commented Apr 5, 2018

Bump. Is this project abandoned?

@ducdigital
Copy link
Author

Hi there @asfktz and @jsw

I am just contributing to this opensource project and have no right to merge this branch to master.

If you use it, just use git format in your package.json like ff:

    "money": "github:ducdigital/money.js",

It seems like the PR is not gonna get merge anytime soon.

@asfktz
Copy link

asfktz commented Apr 6, 2018

Hi, @ducdigital!
How about releasing it as an NPM package of its own?
Something like money2 sounds good to me 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants