Free organizations can connect Slack, but no Slack notifications are ever sent
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 68/100
調査の方向性
Start with AddProjectSlack in src/Exceptionless.Web/Api/Handlers/ProjectHandler.cs and compare its plan checks with SetProjectIntegrationNotificationSettings; then inspect updateSlackNotificationSettings in src/Exceptionless.Web/ClientApp/src/routes/(app)/project/[projectId]/integrations/+page.svelte. Trace how the existing webhook flow handles plan-limit responses and confirm the Slack connection is rejected before its token is stored. Done means Free plan users see the upgrade dialog on both connection and settings changes.
索引モデルが issue の本文から書いたものです。
説明
A project in a Free plan organization can connect Slack, and the app reports success. Notifications are never queued for organizations without premium features, so the Slack channel never receives anything, and nothing in the UI says why. If the user changes the Slack notification switches afterwards, they get a generic error instead of the upgrade dialog.
Where
- Connecting Slack has no plan check: ProjectHandler.cs#L410-L441
- Updating Slack settings does check the plan and returns
426: ProjectHandler.cs#L305-L310 - Notifications are skipped for non-premium organizations: 070_QueueNotificationAction.cs#L30-L32
- The integrations page handles the upgrade response for connecting but not for settings changes: integrations/+page.svelte#L86-L138
Steps to reproduce
- In an organization on the Free plan, open a project's Integrations page and click Connect Slack.
- Complete the Slack authorization. The app shows "Successfully connected Slack integration" and default Slack settings are saved.
- Send a new error to the project. Nothing arrives in Slack.
- Toggle any Slack notification setting. The app shows "Error updating Slack notification settings. Please try again." because the
426isn't handled.
Expected
Connecting Slack on the Free plan returns the plan-limit error (426) before the OAuth token is stored, and the page shows the upgrade dialog, as it already does for webhooks. Settings changes also show the upgrade dialog instead of a generic error.
Suggested fix
- In
AddProjectSlack, load the organization and return the samePlanLimitvalidation result used inSetProjectIntegrationNotificationSettingswhen!organization.HasPremiumFeatures. Ideally check this before exchanging the code, or revoke the token afterwards. - In
updateSlackNotificationSettings, callshowBillingDialogOnUpgradeProblem. - Optionally show a notice on the Slack section for organizations already connected on the Free plan.
- 主要言語
- C#
- スター
- 2.5k
- フォーク
- 508
- 平均マージ
- 2日 17時間
- マージ済み PR（30日）
- 40
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
exceptionless/Exceptionless のほかの issue
-
bug documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
exceptionless/Exceptionless#2707 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
exceptionless/Exceptionless#2706 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
exceptionless/Exceptionless#2705 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
exceptionless/Exceptionless#2704 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
exceptionless/Exceptionless#2703 ·
メンテナーはふだん 1 日以内に返信
exceptionless/Exceptionless の issue をすべて見る
似ている issue
-
possible bug
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
-
Include host.wslc.internal in the HTTPS development certificate対応中かも @afscrome が今日担当しました。 オープンarea-commandlinetools
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
dotnet/aspnetcore#70122 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
:1st_place_medal: good first issue :cowboy_hat_face: refactor :roller_coaster: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ApplETS/Backend-Hello#78 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
メンテナーはふだん 2 日以内に返信
-
status/triage
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Altinn/altinn-studio#21642 ·
メンテナーはふだん 1 日以内に返信