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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add conference registration types CRUD + Public view #4408
Conversation
d196d1e
to
05b8fa8
Compare
@decidim/lot-core ready to review :) |
...conferences/app/controllers/decidim/conferences/admin/conference_registrations_controller.rb
Outdated
Show resolved
Hide resolved
...ences/app/controllers/decidim/conferences/admin/registration_type_publications_controller.rb
Outdated
Show resolved
Hide resolved
...ences/app/controllers/decidim/conferences/admin/registration_type_publications_controller.rb
Outdated
Show resolved
Hide resolved
decidim-conferences/app/controllers/decidim/conferences/admin/registration_types_controller.rb
Outdated
Show resolved
Hide resolved
|
||
def conference_meetings(registration_type) | ||
meeting_components = registration_type.conference.components.where(manifest_name: "meetings") | ||
Decidim::ConferenceMeeting.where(component: meeting_components).where(id: @form.attributes[:conference_meeting_ids]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's a Decidim::ConferenceMeeting
? Shouldn't it be scoped to Decidim::Conferences
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe yes, but 'Decidim::ConferenceMeeting' is already in master. If 'ConferenceMeeting' should be scoped to 'Decidim::Conferences', in my opinion is out of this scope and it will be better to do in another PR.
What you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fair, let's do it in another PR!
c495eaf
to
93cb8c7
Compare
@oriolgual requested changes applied 馃槃 |
馃帺 What? Why?
This PR adds a selective registration by registration types configur days. --> 1st Iteration #4048
Distinguish early and late registration
馃搶 Related Issues
馃搵 Subtasks
CHANGELOG
entry馃摲 Screenshots (optional)