-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Link to persisted queries in tutorial #4359
Conversation
Oops. Looks like this also includes #4355. |
Ah, sorry! Will fix it 😊 |
So, there are more problems with tutorial later on. Should I create one bigger PR or several small ones? |
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Multiple small PRs is generally better. |
Looks like GitHub is reporting merge conflicts. Would you mind rebasing? |
ping @robinpokorny waiting on rebase |
367789e
to
ca1a72b
Compare
ca1a72b
to
96ae953
Compare
Fixed :) |
@captbaritone has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@captbaritone merged this pull request in 007eff0. |
Use correct name and add link to Persisted queries