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

Change intro wording #175

Merged
merged 2 commits into from Sep 20, 2019
Merged

Change intro wording #175

merged 2 commits into from Sep 20, 2019

Conversation

tomer-stripe
Copy link
Collaborator

Reviewers

r? @ob-stripe @brandur-stripe @romain-stripe @mg-stripe
cc @stripe/dev-platform

Copy link
Contributor

@ob-stripe ob-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, but not approving yet to give others a chance to take a look too.

@ob-stripe ob-stripe removed their assignment Sep 18, 2019
@brandur-stripe
Copy link

Seems fine to me too.

@romain-stripe
Copy link

romain-stripe commented Sep 19, 2019

Easily test your webhook integration, tail your logs to monitor requests coming in, or create samples to learn about different Stripe integrations.

I'd change the wording a bit for the last part since users don't create samples. Maybe something like this? Happy to give it another pass!

Interact with any part of the API, debug your integration with real-time request logs, receive webhook events locally with a built-in proxy, and fetch samples to find building blocks for your integration.

@mg-stripe
Copy link
Contributor

My suggestion:

Use the Stripe API from the command line, debug your Stripe integration with real-time logs, forward webhook events to your local application, and quickly get started with pre-built samples (https://stripe.dev/samples).

I think we should include the link so that developers can see what Samples are, but I would be OK with excluding that.

@tomer-stripe
Copy link
Collaborator Author

Thanks all! Updating with @mg-stripe's suggestion :D

@tomer-stripe tomer-stripe merged commit e980986 into master Sep 20, 2019
@tomer-stripe tomer-stripe deleted the tomer/update-intro branch September 20, 2019 16:57
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.

None yet

5 participants