Closed
Bug 1121746
Opened 9 years ago
Closed 9 years ago
Use RequireJS to modularize fxpay and improve tests
Categories
(Marketplace Graveyard :: Payments/Refunds, defect, P5)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kumar, Assigned: scolville)
References
Details
(Whiteboard: [qa-])
If we add RequireJS to the fxpay JS lib we'll unlock the following possibilities: - other apps (like Fireplace) can use un-minified fxpay source which will be great for development - the fxpay test suite can use some really nice isolation patterns. See https://achudnovskij.wordpress.com/2014/09/09/require-js-mock-injection-review-options-and-rfactory-plugin-introduction/ - the fxpay minification process should be more straight forward - people can run fxpay from the source repository more easily - the example app can work with un-minified source
Reporter | ||
Updated•9 years ago
|
Whiteboard: [qa-]
Updated•9 years ago
|
Priority: -- → P5
Reporter | ||
Comment 1•9 years ago
|
||
digitarald suggests that we may be able to get node + browser support with a UMD pattern: http://davidbcalhoun.com/2014/what-is-amd-commonjs-and-umd/ and maybe rewire could be used for mock injection: https://www.npmjs.com/package/rewire-webpack See also: https://github.com/jhnns/rewire
Reporter | ||
Comment 2•9 years ago
|
||
Stuart got this ^5 https://github.com/mozilla/fxpay/pull/102
Assignee: kumar.mcmillan → scolville
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•