Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Link to nonexistent schedule #3136

Open
1 of 2 tasks
AndrewKvalheim opened this issue Mar 10, 2023 · 4 comments
Open
1 of 2 tasks

Link to nonexistent schedule #3136

AndrewKvalheim opened this issue Mar 10, 2023 · 4 comments
Labels
bug 🐛 Something is wrong and needs to be fixed good first issue For RoR beginners or people unfamiliar with the application

Comments

@AndrewKvalheim
Copy link
Member

AndrewKvalheim commented Mar 10, 2023

I'm submitting a…

  • Bug Report
  • Feature Request

Current behavior

The home page can display a link to the schedule when there is no schedule yet:

screenshot

Following the link triggers a server error:

Started GET "/conferences/example/schedule"
Processing by SchedulesController#show as HTML
  Parameters: {"conference_id"=>"example"}
  Rendering layout layouts/application.html.haml
  Rendering schedules/show.html.haml within layouts/application
  Rendered schedules/_schedule_tabs.html.haml
  Rendered schedules/_carousel.html.haml
  Rendered schedules/show.html.haml within layouts/application
  Rendered layout layouts/application.html.haml
Completed 500 Internal Server Error

ActionView::Template::Error (undefined method `each' for nil:NilClass):
    22:                   = (td_start_time).strftime("%M")
    23:               - td_start_time += @step_minutes
    24:           - start_room_time = start_time
    25:           - @rooms.each do |room|
    26:             - start_room_time = start_time
    27:             - span = 1
    28:             %tr

app/views/schedules/_carousel.html.haml:25
app/views/schedules/_carousel.html.haml:11
app/views/schedules/show.html.haml:22
app/views/schedules/show.html.haml:18

Expected correct behavior

When no selected schedule exists:

  • No link is displayed.
  • Visiting the expected address of the schedule returns 404 Not Found, not an error message.

i.e. the setting Show Schedule on the home and splash page functions as an allowance, not as an unchecked manual control.

Steps to reproduce

  1. As an admin:
    1. Create a conference.
    2. At ManageProgramEdit, enable Show Schedule on the home and splash page.
  2. As an unauthenticated viewer:
    1. Visit the home page.
    2. Click Schedule.

Other information

Other functionality such as notify_on_schedule_public? may be similarly affected; needs review.

@hennevogel hennevogel added bug 🐛 Something is wrong and needs to be fixed good first issue For RoR beginners or people unfamiliar with the application labels Apr 19, 2023
@enterbrianup
Copy link

i am interested in solving this issues.

@hennevogel
Copy link
Member

hennevogel commented Jul 18, 2023

i am interested in solving this issues.

We are interested to see your pull request. If you need anything from us, let us know 😃

@Yash-Singh-Pathania
Copy link

Working on this @enterbrianup did you finish up this cant see anything as such let me pick this up .
@hennevogel when can i expect the pull request to be reviewed ?

@hennevogel
Copy link
Member

@Yash-git-hub whenever I find time in my personal schedule :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something is wrong and needs to be fixed good first issue For RoR beginners or people unfamiliar with the application
Projects
None yet
Development

No branches or pull requests

4 participants