flyblade_controller.rb

class FlybladeController < ApplicationController
  def check_terms_of_service_version
    return if request.path == edit_user_path(current_user)

    version = Flyblade::TermsOfServicePolicy.current_version
    unless current_user.terms_of_service_version == version
      return redirect_to edit_user_path(current_user)
    end
  end
end

I think you can omit line 3 if you just skip the method for the edit user action. skip_before_action :check_terms_of_service_version, only: [:edit]

What do you think of the following?

• Add validates_acceptance_of: terms_of_service to User
• Add terms_of_service field to the new user form
• When a new user is saved, persist the terms of service based on Flyblade::TermsOfServicePolicy.current_version
• Have a before_action that checks the user has accepted most recent terms of service (if not, redirect to standalone terms of service form)

skip_before_action :check_terms_of_service_version, only: [:edit, :update]

Needed on update as well right?