Skip to content
This repository has been archived by the owner on Feb 27, 2021. It is now read-only.

Dev notes for master branch. #32

Open
5 tasks
futhr opened this issue Dec 10, 2014 · 0 comments
Open
5 tasks

Dev notes for master branch. #32

futhr opened this issue Dec 10, 2014 · 0 comments

Comments

@futhr
Copy link
Contributor

futhr commented Dec 10, 2014

After some refactor on master branch I noticed a lot that can be improved:

  • Change currency display wrong price format due to Spree Core has general settings options for this based on single currency usage. This should be changed in Core to fetch format from Money.
  • Automatic update functionality like optional automatic conversion from base currency. #16. This functionality also exist in a diverted repository https://github.com/pronix/spree-multi-currency but no clue on its state.
  • Validate that general config settings support multiple stores.
  • Ensure it show per default correct currency related to language, this is probably harder then it sounds and probably need to add default currency value to countries. Or maybe just Money::Currency.find_by_iso_numeric will do the trick.
  • Need to look into issue Bind Payment Methods to Currency #23

These are dev memory notes, you welcome to help out with PR if you find it valid.

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

No branches or pull requests

1 participant