Skip to content

add riceathon scenario#203

Open
System-End wants to merge 2 commits into
hackclub:mainfrom
System-End:riceathon-scenario
Open

add riceathon scenario#203
System-End wants to merge 2 commits into
hackclub:mainfrom
System-End:riceathon-scenario

Conversation

@System-End
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings March 13, 2026 18:50
@System-End
Copy link
Copy Markdown
Contributor Author

@24c02 Any changes?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new “Riceathon” onboarding scenario and introduces an opt-in mechanism to automatically promote users to full Slack members upon identity verification approval.

Changes:

  • Add riceathon to the shared Slack channel config.
  • Introduce promote_on_verification scenario flag (default false) and a new OnboardingScenarios::Riceathon scenario opting in.
  • Add an auto-promotion hook in backend verification approval to enqueue Slack promotion.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

File Description
config/slack_channels.yml Adds the riceathon channel ID so scenarios can reference it via chans(...).
app/models/onboarding_scenarios/riceathon.rb New onboarding scenario defining fields, Slack channels, and opting into verification-based promotion.
app/models/onboarding_scenarios/base.rb Adds the promote_on_verification flag with a default of false.
app/controllers/backend/verifications_controller.rb Enqueues an auto-promotion job when a verification is approved for opted-in scenarios.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/controllers/backend/verifications_controller.rb Outdated
Comment thread app/models/onboarding_scenarios/riceathon.rb
@24c02
Copy link
Copy Markdown
Member

24c02 commented Mar 16, 2026

can you explain the rationale for promote_on_verification more

@24c02
Copy link
Copy Markdown
Member

24c02 commented Mar 16, 2026

tutorial agree seems like a questionable way to do it

@System-End
Copy link
Copy Markdown
Contributor Author

can you explain the rationale for promote_on_verification more

this program will be advertised on very public places with peiople of all age ranges, so i dont want people to have full slack access save for riceahton help chan and idv help until verified

@System-End
Copy link
Copy Markdown
Contributor Author

tutorial agree seems like a questionable way to do it

any sugestions on how else i should?

@24c02
Copy link
Copy Markdown
Member

24c02 commented Mar 16, 2026

look at what tutorialagreejob does?

@System-End
Copy link
Copy Markdown
Contributor Author

look at what tutorialagreejob does?

hows this?

@System-End
Copy link
Copy Markdown
Contributor Author

cc @maxwofford since ur the sponsor

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.

3 participants