Skip to content

Fix nil error in confirmation_token url helper#901

Merged
manno merged 2 commits intomainfrom
fix-confirmation-token-url-helper
Jul 7, 2022
Merged

Fix nil error in confirmation_token url helper#901
manno merged 2 commits intomainfrom
fix-confirmation-token-url-helper

Conversation

@manno
Copy link
Member

@manno manno commented Jul 7, 2022 

  No route matches {:action=>"confirm", :conference_acronym=>"CONFERENCE", :controller=>"cfp/events", :id=>ID, :locale=>"en", :token=>nil}, possible unmatched constraints: [:token]
Did you mean?  cfp_event_confirm_by_token_url
  app/views/cfp/people/_table.html.haml:37

This is related to the partly fix from #817
Conference may have two "event_people" objects for the same "person", one as "submitter", another as "speaker". Only the "speaker" has the confirmation token.

manno added 2 commits July 8, 2022 00:02
@manno manno merged commit b6781ae into main Jul 7, 2022
@manno manno deleted the fix-confirmation-token-url-helper branch July 7, 2022 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@manno