-
Notifications
You must be signed in to change notification settings - Fork 59
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
docs: remove SummitCallouts #534
docs: remove SummitCallouts #534
Conversation
✅ Docs Preview ReadyConfiguration{
"repoOverrides": {
"apollographql/apollo-ios-dev@main": {
"remote": {
"owner": "Meschreiber",
"repo": "apollo-ios-dev",
"branch": "ms/remove-summit-callouts"
}
}
}
}
2 pages published. Build will be available for 30 days. |
✅ Deploy Preview for apollo-ios-docc ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for eclectic-pie-88a2ba ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@Meschreiber - we can ignore the failed "Check PR Metadata" check but there are also failed Netlify checks; do we need to be concerned about those? I did check the modified pages (index and get-started) in the deploy preview and they both render correctly so not sure what the failed Netlify checks are about. |
Thanks @calvincestari ! Those Netlify checks are for the legacy docs site--I'll remove the associated |
Hmm, I think those checks should go away with future PRs, but will look into if they don't. In the meanwhile, would you mind merging when you get a chance please? I don't have write access for the repo. TIA! |
Removes callouts for iOS Summit sessions.