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

Catch 2-4-stable up with Master #94

Open
wants to merge 42 commits into
base: 2-4-stable
Choose a base branch
from

Conversation

davekiss
Copy link
Contributor

This merges in the changes that have occurred on Master over the past few months.

Also, I do believe that line_items.digital? should report as digital if the master variant is digital. That way, a user can be issued download links from the master variant, regardless of if their purchased variant is digital or not.

This would allow you to send out any common downloads via the master variant, and then send out variant-specific downloads with each variant.

future and others added 30 commits December 2, 2014 11:06
This conversion is done by Transpec 3.1.0 with the following command:
    transpec

* 71 conversions
    from: obj.should
      to: expect(obj).to

* 24 conversions
    from: == expected
      to: eq(expected)

* 10 conversions
    from: obj.should_receive(:message)
      to: expect(obj).to receive(:message)

* 8 conversions
    from: lambda { }.should
      to: expect { }.to

* 4 conversions
    from: obj.should_not
      to: expect(obj).not_to

* 3 conversions
    from: it { should ... }
      to: it { is_expected.to ... }

* 2 conversions
    from: obj.should_not_receive(:message)
      to: expect(obj).not_to receive(:message)

* 1 conversion
    from: lambda { }.should_not
      to: expect { }.not_to

* 1 conversion
    from: obj.should_receive(:message).and_return { value }
      to: obj.should_receive(:message) { value }

For more details: https://github.com/yujinakayama/transpec#supported-conversions
This conversion is done by Transpec 3.1.0 with the following command:
    transpec

* 6 conversions
    from: obj.stub(:message => value)
      to: allow(obj).to receive_messages(:message => value)

For more details: https://github.com/yujinakayama/transpec#supported-conversions
@davekiss
Copy link
Contributor Author

@halo or @JDutil, mind having a look?

@JDutil
Copy link
Member

JDutil commented Sep 15, 2015

The changes look okay to me, but I haven't manually tested things out.

@davekiss
Copy link
Contributor Author

@JDutil pushing to staging app now, will report back

@davekiss
Copy link
Contributor Author

@JDutil works like a charm on my end

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.

5 participants