[Chore] Remove API crawl script #19454
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kodster28
requested review from
kristianfreeman,
pedrosousa,
haleycode,
maxvp,
marciocloudflare,
GregBrimble,
KianNH,
WalshyDev and
a team
as code owners
KianNH
approved these changes
Jan 27, 2025
github-actions
bot
assigned GregBrimble, haleycode, KianNH, kristianfreeman, marciocloudflare, maxvp, pedrosousa and WalshyDev
Deploying cloudflare-docs with
|
| Latest commit: |
4c41f4a
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9d341776.cloudflare-docs-7ou.pages.dev |
| Branch Preview URL: | https://remove-api-crawl-script.cloudflare-docs-7ou.pages.dev |
deadlypants1973
pushed a commit
that referenced
this pull request
Jan 29, 2025
* [Chore] Remove API crawl script * remove command from package.json
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
When our old API docs site was using Stoplight, we had two interconnected issues that made it difficult to maintain API docs links w/in our docs.
deprecatedstatus would also change the endpoint URL.With the migration to a new API docs framework, these issues should be gone and we can remove this code from our repo.
Problem should now be solved by either standard SEO crawling or the work that @bllchmbrs is helping us with.
Risks
None. This only involved a scheduled GitHub action, so there should be no impact to our site.