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

Release v11.5.0 #835

Merged
merged 4 commits into from
Nov 9, 2017
Merged

Release v11.5.0 #835

merged 4 commits into from
Nov 9, 2017

Conversation

danj-stripe
Copy link
Contributor

  • Update README (if necessary)
  • Update CHANGELOG with any new features or breaking changes
  • Update MIGRATING (if necessary)
  • Update STPSDKVersion in STPAPIClient.h
  • Update version in Info.plist (make sure to not change the "InfoDictionary" field)
  • Update VERSION
  • Update Stripe.podspec (make sure there aren't any new directories added, and if so, add them to the "files" section
  • Update documentation: ./ci_scripts/build_documentation.sh
  • Make sure pod lib lint succeeds without warnings or errors
    • There is a single, expected, warning, due to #823
  • Make sure ./ci_scripts/export_builds.sh succeeds without warnings or errors

* Update README:
  * We support building with either Xcode 8 or 9
  * Use example backend app at the tip of master, instead of linking to a specific tag.
* Update CHANGELOG
* Update `STPSDKVersion`
* Update versions in `Info.plist`
* Update `VERSION`
* Update `Stripe.podspec`
README.md Outdated
@@ -49,7 +50,7 @@ To try out any of these examples, clone this repository, open `Stripe.xcworkspac
1. Execute `./setup.sh` from the root of the repository to build the necessary dependencies.
2. If you haven't already, sign up for a [Stripe account](https://dashboard.stripe.com/register) (it takes seconds). Then go to https://dashboard.stripe.com/account/apikeys.
3. Replace the `stripePublishableKey` constant in CheckoutViewController.swift (for the Standard Integration app) or Constants.m (for the Custom Integration app) with your Test Publishable Key.
4. Head to https://github.com/stripe/example-ios-backend/tree/v11.0.0 and click "Deploy to Heroku" (you may have to sign up for a Heroku account as part of this process). Provide your Stripe test secret key for the STRIPE_TEST_SECRET_KEY field under 'Env'. Click "Deploy for Free".
4. Head to https://github.com/stripe/example-ios-backend and click "Deploy to Heroku" (you may have to sign up for a Heroku account as part of this process). Provide your Stripe test secret key for the STRIPE_TEST_SECRET_KEY field under 'Env'. Click "Deploy for Free".
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you remove the versioned link from the backend url?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ksun-stripe made updates to the example-ios-backend, and that tag isn't current any more. We could also re-tag it and update the link to the new tag, but I wasn't sure why we wouldn't just want to link people to master

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah we specifically wanted to start making tagged releases for the backend and having the tagged releases here match up to the version of the backend that supports this version of the sdk. Otherwise you can get confusing version mismatches (eg latest master on backend might not match the latest tagged release of the sdk. Or if you are on an old version you can't easily figure out what version of the backend you need).

If Kathy has made new changes to the backend that we want people to be using with 11.5.0 we should create a new tagged version for the backend and link that here instead.

The latest changes were for Android support, and the iOS app continues to work fine with
v11.0.0 of the backend.
@bdorfman-stripe
Copy link
Contributor

LGTM

@bdorfman-stripe bdorfman-stripe merged commit 243da13 into master Nov 9, 2017
@danj-stripe danj-stripe deleted the danj-release-v11.5.0 branch December 12, 2017 22:26
davidme-stripe pushed a commit that referenced this pull request Mar 7, 2022
* Remove separator

* Display the menu on top of the active digit view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants