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

Link to persisted queries in tutorial #4359

Conversation

robinpokorny
Copy link
Contributor

Use correct name and add link to Persisted queries

@captbaritone
Copy link
Contributor

Oops. Looks like this also includes #4355.

@robinpokorny
Copy link
Contributor Author

Ah, sorry! Will fix it 😊

@robinpokorny
Copy link
Contributor Author

So, there are more problems with tutorial later on. Should I create one bigger PR or several small ones?

@facebook-github-bot
Copy link
Contributor

@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@captbaritone
Copy link
Contributor

Multiple small PRs is generally better.

@captbaritone
Copy link
Contributor

Looks like GitHub is reporting merge conflicts. Would you mind rebasing?

@captbaritone
Copy link
Contributor

ping @robinpokorny waiting on rebase

@robinpokorny robinpokorny force-pushed the Link-to-persisted-queries-in-tutorial branch from 367789e to ca1a72b Compare July 27, 2023 07:39
@robinpokorny robinpokorny force-pushed the Link-to-persisted-queries-in-tutorial branch from ca1a72b to 96ae953 Compare July 27, 2023 07:42
@robinpokorny
Copy link
Contributor Author

Fixed :)

@facebook-github-bot
Copy link
Contributor

@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@captbaritone merged this pull request in 007eff0.

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

Successfully merging this pull request may close these issues.

3 participants