PRDCT-348: mark Dark Sky deprecated — the upstream API shut down in 2023 - #1082
Merged
Conversation
…2023 The June audit listed dark-sky as the only outright Kill in the whole corpus. The page is still live and still reads as a working connector. PR #975, the PR for this very category, does not touch it. Verified from the source, not from the audit note: darksky.net/dev now serves an Apple support page stating "Support for the Dark Sky API ended on March 31, 2023, and has been replaced by Apple's WeatherKit API." The component keboola.dark-sky-augmentation is flagged `deprecated` and `excludeFromNewList` in the Storage component index, so it cannot be added to a project either. So the page was actively misleading: it told readers to "create an account in the Dark Sky API" and promised "1,000 API calls a day for free" -- neither is possible. - Added a `:::caution[Deprecated]` notice with the shutdown date, the reason, and what to do about an existing configuration. - Pointed affected users at the Weather API connector, which carries no deprecated/excludeFromNewList flags and covers forecast, real-time, and historical weather. - Rewrote the sign-up instructions and the dead API-docs link in the past tense rather than leaving instructions nobody can follow. - Flagged it in the "Other Data Source Connectors" list too, where it had been described as if it worked. Deliberately NOT deleted. The audit's verdict was Kill + 301, but the component is deprecated rather than removed, so a customer with a failing configuration still needs to find out why -- a redirect to the category index would tell them nothing. Whether to delete the page outright is an owner call; the evidence is here and the notice is safe either way. Verified: build clean (306 pages); audit-phase2 at 45 broken links / 0 missing images, unchanged from main; no findings on either touched page. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jordanrburger
marked this pull request as ready for review
August 5, 2026 16:24
jordanrburger
approved these changes
Aug 5, 2026
jordanrburger
enabled auto-merge
August 5, 2026 16:25
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 2/5) · profile docs
Well-crafted deprecation rewrite needs a human to confirm the product-behaviour claim; no skill flagged anything.
Suggested reviewers: @keboola/docs
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
The June audit listed
dark-skyas the only outright Kill in the entire corpus. The page is still live, still reads as a working connector — and #975, the PR for this very category, does not touch it.Verified from the source, not from the audit note
darksky.net/devnow serves an Apple support page: "Support for the Dark Sky API ended on March 31, 2023, and has been replaced by Apple's WeatherKit API." Andkeboola.dark-sky-augmentationis flaggeddeprecatedandexcludeFromNewListin the Storage component index, so it cannot be added to a project either.The page was therefore actively misleading. It told readers to "create an account in the Dark Sky API" and promised "Dark Sky offers 1,000 API calls a day for free" — neither is possible, and the linked API docs no longer exist.
What changed
:::caution[Deprecated]notice carrying the shutdown date, the reason, and what to do about an existing configuration.deprecated/excludeFromNewListflags, and it covers forecast, real-time, and historical weather.Deliberately NOT deleted — an owner call
The audit's verdict was Kill + 301. I did not do that, on purpose: the component is deprecated, not removed, so a customer whose configuration is failing still needs to find out why. A redirect to the category index would tell them nothing.
If you want the page gone, the delete is mechanical from here (drop the page, add the redirect, remove the nav entry, fix the one inbound link in
other/index.md). This notice is the safe state either way.Verification
Build clean, 306 pages.
audit-phase2at 45 broken links / 0 missing images, unchanged frommain; no findings on either touched page.Third PR from the reconciliation of the June audits against the live sites — companions #1080 and #1081.