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

Add support for RefreshURL and ReturnURL on AccountLink #1124

Merged
merged 1 commit into from
Jun 18, 2020

Conversation

remi-stripe
Copy link
Contributor

@remi-stripe remi-stripe commented Jun 16, 2020

r? @cjavilla-stripe
cc @stripe/api-libraries

@remi-stripe remi-stripe changed the title [WIP] Add support for RefreshURL and ReturnURL on AccountLink Add support for RefreshURL and ReturnURL on AccountLink Jun 18, 2020
Copy link
Contributor

@cjavilla-stripe cjavilla-stripe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@remi-stripe remi-stripe merged commit 1cfb7c4 into master Jun 18, 2020
@remi-stripe remi-stripe deleted the remi-account-link branch June 18, 2020 21:19
nadaismail-stripe pushed a commit that referenced this pull request Oct 18, 2024
…e subscription (#1124)

* Add evergreen as a subscription type constant

* Utilize original create salesforce order for evergreen

* Edit comment and fix up implementation of create evergreen order

* Created evergreen order test file with test skeleton

* Added test for if end date is nil

* Accessed quote obejct from order and wrote more tests

* Fixed rubocop errors

* Added todo for getting quote, requires further investigation

* Add function to check subscription type of an order

* Add evergreen constant and add check in order

* Write test to simulate translating evergreen order

* Raise user error when evergreen encountered and test

* Create test to simulate evergreen order translate

* Undo changes from wrong task

* Create test to simulate evergreen order translate without throwing error

* Rubocop fix

* Rubocop again

* Remove lock file changes

* remove spaces

* Add some logic for creating subscription

* Subscriptions can be created, fixed payment missing

* Added days until due for invoice based on sf order

* Update sf order with subscription id

* Added assertions for translated stripe subscription

* Add time test to ensure invoices get created each pay period

* Add comments for another test

* Repond to PR comments

* Add test for different pay period length

* Tested longer pay period test

* Remove subscription from user field mappings

* Add Jira for sanitize stripe subscription

* Remove redundant email

* Lint and remove dump

* Remove changes to lock file

* Started test for subscription term not 1

* Changed error message to not yet supported

* Changed method of checking whether order evergreen

* Added test for user error when order has both evergreen and renewable

* Add jira for modifying add_product_to_cpq_quote function

* Linted

* Remove ruby from lock file

* Move create evergreen sf order function back

* Add test and check for whether evergreen order subscription term is 1

* Lint

* Add two products to cpq quote

* Wrote multi order item translate test

* Improved user field defaults email creation

* Bye bye ruby

* Create random email in field defaults instead of mapping

* Update Gemfile.lock

* Lint

* Lint

* Update Gemfile.lock

* Removed get quote helpful function

* Gemfile

* Remove redundant fields from subscription

* Put subscription code back in migrate pre integration order

* Remove test that throws error on evergreen orders

* Catch renewable/evergreen salesforce products

* Write order creation experiment function in test

* Tests pass w this version

* Removed redundant code for create subscription

* Lint

* Update Gemfile.lock

* Remove experimentation function from test

* Update Gemfile.lock

* User email changes

* Implemented mapper for translate

* Add user field defaults to evergreen tests

* Remove Renewable/Evergreen code

* Edit

* Update Gemfile.lock

* Lint

* Add evergreen subscription term constant

* Add more asserts in basic evergreen order translate and metadata

* Remove invoice test

* Fixed test

* Update Gemfile.lock

* Fix test

* Lint

* Use constant

* Use interface in new test

* Fix the tests
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.

2 participants