Skip to content

Conversation

@caseyvaughn
Copy link
Collaborator

@caseyvaughn caseyvaughn commented Sep 12, 2025

  • This work is part of Create Facilitator Model #88

  • Create Facilitator model

  • Setup relationship between User & Facilitator (user belongs_to :facilitator & facilitator has_one :user)

  • Setup basic CRUD & form for Facilitator (new, edit, show, index, destroy)

@caseyvaughn caseyvaughn force-pushed the create-facilitator-model branch 3 times, most recently from 1ad2c59 to 42c08b7 Compare September 12, 2025 18:45
@caseyvaughn caseyvaughn marked this pull request as ready for review September 12, 2025 18:46
Copy link
Collaborator

@maebeale maebeale left a comment

Choose a reason for hiding this comment

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

exciting! great work, @caseyvaughn !

# Skip login requirement for new facilitator form
# temporarily skipping authentication for all actions for development ease
#TODO: remove :index & :show from skip_before_action
skip_before_action :authenticate_user!, only: [:index, :show, :update, :edit, :new, :create]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe consider an ... if Rails.env.development? so this doesn't slip through to production?

Copy link
Collaborator

@johnpaulashenfelter johnpaulashenfelter left a comment

Choose a reason for hiding this comment

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

All this seems 👍

@caseyvaughn caseyvaughn force-pushed the create-facilitator-model branch from f600fd7 to 573c3bd Compare September 12, 2025 19:36
@caseyvaughn caseyvaughn merged commit c5108ec into main Sep 12, 2025
2 checks passed
@caseyvaughn caseyvaughn deleted the create-facilitator-model branch September 12, 2025 19:38
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.

4 participants