Skip to content

ci: migrate slack notifications + linkedin troubleshooting rework#145

Merged
MarketDataApp merged 3 commits into
mainfrom
staging
Apr 15, 2026
Merged

ci: migrate slack notifications + linkedin troubleshooting rework#145
MarketDataApp merged 3 commits into
mainfrom
staging

Conversation

@MarketDataApp

Copy link
Copy Markdown
Owner

Summary

  • Migrate all 4 Slack failure notifications (deploy-docs.yml, pr-checks.yml ×2, post-deploy-tests.yml) from the direct webhook (SLACK_ALERTS_WEBHOOK_URL) to the centralized notify service (SLACK_NOTIFY_URL + SLACK_NOTIFY_TOKEN), matching the pattern used in marketdata-amember.
  • Rework LinkedIn troubleshooting page to cover all profile issues (not just "missing").

Test plan

  • deploy-docs.yml completes successfully on merge
  • post-deploy-tests.yml canonical check for /docs/account/troubleshooting/linkedin-missing/ passes once prod has the renamed linkedin-issues page
  • Slack failure notifications route correctly (can verify by inducing a failure later, or trust the amember parity)

Rename linkedin-missing to linkedin-issues and broaden the article beyond
missing profiles to cover name mismatches, employment mismatches, and
other common LinkedIn verification failures. Adds client-side redirect
from the old path.
@socket-security

socket-security Bot commented Apr 15, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedreact@​18.3.11001008497100
Addedtypescript@​4.9.5100100909990
Addedreact-dom@​18.3.1921009298100

View full report

@MarketDataApp MarketDataApp merged commit 4a86963 into main Apr 15, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant