-
-
Notifications
You must be signed in to change notification settings - Fork 160
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
[16.0][MIG] website_event_questions_by_ticket: Migration to version 16.0 #344
[16.0][MIG] website_event_questions_by_ticket: Migration to version 16.0 #344
Conversation
This module allows to set that an event question is only asked for certain tickets. If the question is general, then having one of the allowed tickets will serve for showing it. If the question is specific to each attendee, then the evaluation will be done for each of the tickets. Configuration ============= 1. Go to *Events > Events* 2. Create or select one event. 3. On the "Questions" page, create a new question or edit an existing one. 4. Fill the field "Limited to tickets" for restricting the question to that ticket. Leave it blank for making it available for all tickets. Usage ===== 1. Go to the website. 2. Go to "Events" section and select one of them. 3. Select ticket quantity and click on "Register". 4. On the popup dialog, the questions linked to that ticket and those without restrictions will be shown. Known issues / Roadmap ====================== * Add tests (tour) to the module.
[MIG] website_event_questions_by_ticket: Migration to 16.0
/ocabot migration website_event_questions_by_ticket |
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.
Code review and tested on runboat.
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.
Tested functionallity. LGTM!
/ocabot merge nobump |
What a great day to merge this nice PR. Let's do it! |
Congratulations, your PR was merged at 4043142. Thanks a lot for contributing to OCA. ❤️ |
@Tecnativa
TT45239
@pedrobaeza