Skip to content
This repository has been archived by the owner on Oct 1, 2019. It is now read-only.

Add order invoice property to submitTransaction and authorizeTransaction #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kylecannon
Copy link

This doesn't break backwards compatibility with anything due to adding in the property via extending the Order Model.

Example:

    var Order = require('42-cent-model').Order;
    var OrderProperties = {
        invoiceNumber: 'AAAAAA-1234555'
    };

    var Order = new Order(OrderProperties);

@lorenzofox3
Copy link
Contributor

it seems to break the build

@kylecannon
Copy link
Author

I couldn't get the tests to pass at all even without my changes present. Do you want to run the tests without my changes and see if the tests pass on your end?

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

Successfully merging this pull request may close these issues.

2 participants