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.

Sign up for GitHub

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

Jump to bottom

Agenda and My Schedule #56

Merged
merged 42 commits into from
Aug 15, 2024
Merged

Agenda and My Schedule #56

merged 42 commits into from
Aug 15, 2024

Conversation

LuigiR0jas
Copy link
Collaborator

Description

Please include a summary of the change. Also, include any additional information that you think is important for reviewers to know. Link to a related issue if applicable.

How has this been tested?

Please mark the tests that you ran to verify your changes. If difficult to test, consider providing instructions so reviewers can test.

  • Manual testing
  • System tests
  • Unit tests
  • None

Checklist

  • CI pipeline is passing
  • My code follows the conventions of this project
  • I have performed a self-review of my code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • I have added seed data to the database (if applicable)

Release tasks

Add any tasks that need to be done before/after the release of this feature.

Screenshots/Loom

This section is relevant in case we want to share progress with the team, otherwise, it can be omitted.

@LuigiR0jas LuigiR0jas self-assigned this Aug 7, 2024
@LuigiR0jas LuigiR0jas requested a review from EdgarB August 7, 2024 15:15
@LuigiR0jas LuigiR0jas marked this pull request as ready for review August 7, 2024 15:16
@LuigiR0jas LuigiR0jas changed the title Events index Agenda Aug 7, 2024
@LuigiR0jas LuigiR0jas requested review from andresag4 and removed request for EdgarB August 7, 2024 16:10
Sergio-e
Sergio-e reviewed Aug 9, 2024
View reviewed changes
app/views/schedules/index.html.erb Outdated Show resolved Hide resolved
app/views/schedules/index.html.erb Outdated Show resolved Hide resolved
app/assets/images/placeholder.png Outdated Show resolved Hide resolved
app/models/session.rb Outdated Show resolved Hide resolved
app/controllers/schedules_controller.rb Outdated Show resolved Hide resolved
app/models/conference.rb Show resolved Hide resolved
app/models/session.rb Outdated Show resolved Hide resolved
app/views/sessions/_card.html.erb Outdated Show resolved Hide resolved
app/views/sessions/_card.html.erb Outdated Show resolved Hide resolved
app/controllers/application_controller.rb Show resolved Hide resolved
@LuigiR0jas LuigiR0jas changed the title Agenda Agenda and My Schedule Aug 14, 2024
Sergio-e
Sergio-e reviewed Aug 14, 2024
View reviewed changes
app/controllers/attendees_controller.rb Outdated Show resolved Hide resolved
config/routes.rb Outdated Show resolved Hide resolved
app/controllers/sessions_controller.rb Outdated Show resolved Hide resolved
app/views/sessions/_card.html.erb Outdated Show resolved Hide resolved
app/views/layouts/_flash_message.html.erb Outdated Show resolved Hide resolved
app/views/sessions/_card.html.erb Outdated Show resolved Hide resolved
app/views/sessions/_card.html.erb Outdated Show resolved Hide resolved
app/views/sessions/_filters.html.erb Outdated Show resolved Hide resolved
app/views/sessions/index.html.erb Outdated Show resolved Hide resolved
app/controllers/attendees_controller.rb Show resolved Hide resolved
@Sergio-e Sergio-e mentioned this pull request Aug 14, 2024
Merged
10 tasks
@Sergio-e Sergio-e merged commit 4c892c9 into main Aug 15, 2024
4 checks passed
@Sergio-e Sergio-e deleted the 23-events-index-agenda branch August 15, 2024 02:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sergio-e Sergio-e Sergio-e left review comments

@andresag4 andresag4 andresag4 left review comments

Assignees

@LuigiR0jas LuigiR0jas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@LuigiR0jas @andresag4 @Sergio-e