Override send_password_change_notification to handle accepting invitation

[adzap]

During accepting an invitation we don't want a password change notification on the user's first sign in. This checks the value of accepting_invitation?.

This fixes #679

[scambra]

Thanks

[ACPK]

@scambra - I hit this issue when trying devise_invitable today. Is there a specific feature I need to enable as I currently have "send_password_change_notification" set to true. Hence, users that accepted an invitation got an email that their password was changed.

[scambra]

You don't have to enable anything, it should work by default

You can override send_password_change_notification? in your model to debug if it's returning right value

      def send_password_change_notification?
        super.tap do |value|
          logger.info "DEBUG password change notification #{value.inspect}"
          logger.info "DEBUG accepting invitation? #{accepting_invitation.inspect}"
        end
      end

Expected log is:
DEBUG password change notification false
DEBUG accepting invitation? true