-
-
Notifications
You must be signed in to change notification settings - Fork 332
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
Money type domains WIP #662
Conversation
@mathemancer I thought we were going to leave the multi-currency I made the following issues to build a design/frontend for it: |
It leaves it in, but avoids hooking it up to the API. It's also removed from "supported types", since that would end up getting picked up by the API code. This is the reason for the removal of all the associated tests. |
This is a serious problem -- 90,000 (ish) casting functions are created.
This pull request has not been updated in 45 days and is being marked as stale. It will automatically be closed in 30 days if not updated by then. |
@mathemancer do you want to close this PR since we're going to be implementing money via display options? |
Good idea, closing. |
Fixes #417
Technical details
Screenshots
Checklist
Update index.md
).master
branch of the repositoryvisible errors.
Developer Certificate of Origin
Developer Certificate of Origin