Skip to content

feat(campaign): open the discussion to any logged-in user#4859

Merged
yingshinlee merged 1 commit into
developfrom
feat/campaign-discussion-open-to-all
Jun 17, 2026
Merged

feat(campaign): open the discussion to any logged-in user#4859
yingshinlee merged 1 commit into
developfrom
feat/campaign-discussion-open-to-all

Conversation

@yingshinlee

Copy link
Copy Markdown
Collaborator

放寬活動討論區留言權限:從「只有報名成功者/主辦」改成「任何登入用戶皆可留言」。基本的帳號狀態(banned/frozen)與活動封存 guard 不變。測試已更新(non-participant/pending/rejected 現在都能留言)。base=develop→ICU。

搭配前端會另開一支:canComment 改 viewer.isAuthed、留言點讚 hasUpvote、長留言收回鈕、輸入框框內發布。

Relax the campaign-discussion comment permission from "only succeeded
participants (or organizers)" to any logged-in user. Basic user-state and
campaign-state guards are unchanged. Tests updated: non-participant / pending /
rejected applicants can now comment.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@yingshinlee yingshinlee requested a review from a team as a code owner June 17, 2026 12:25
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.76%. Comparing base (915098c) to head (c2cb191).
⚠️ Report is 39 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #4859      +/-   ##
===========================================
- Coverage    72.93%   72.76%   -0.18%     
===========================================
  Files         1068     1070       +2     
  Lines        21541    21295     -246     
  Branches      4802     4654     -148     
===========================================
- Hits         15712    15496     -216     
+ Misses        5347     5322      -25     
+ Partials       482      477       -5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@yingshinlee yingshinlee merged commit f9a4c17 into develop Jun 17, 2026
5 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