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

[14.0][MIG] event_registration_cancel_reason: Migration to 14.0 #270

Merged

Conversation

LudLaf
Copy link
Contributor

@LudLaf LudLaf commented Apr 21, 2022

Standard migration to version 14.0

@Tecnativa

TT35392

pedrobaeza and others added 4 commits April 21, 2022 20:12
=============================================
Reasons for event registrations cancellations
=============================================

This module allows to add a predefined reason why a registration for an event
is cancelled.

Configuration
=============

Go to *Marketing > Configuration > Events > Registrations cancel reasons* for
setting the possible reasons you want to handle. You can also set if a reason
is only available for certain events or, if you leave it empty, for all.

Usage
=====

Go to *Marketing > Events > Events*, enter into the details or any of the
events, and press the red circle button in the right part of any of the
registrations. A new screen will appear asking you to enter the reason why the
registration is cancelled.

You will get also the same screen when pressing the button
*Cancel registration* from the details view of a registration in
*Marketing > Events > Registrations*. On this object, you can group by the
cancel reason in the list, calendar or graph view.
…ing and fix tests.

Fix OCA#35. When an event had no type, it was not getting in the mapping because of the way Odoo generates recordsets.
- Updated README
- Updated views
- Updated tests
- Improve tests to achieve full test coverage
@rousseldenis
Copy link
Sponsor Contributor

/ocabot migration event_registration_cancel_reason

Copy link
Sponsor Contributor

@rousseldenis rousseldenis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review. Nitpicking

@pedrobaeza pedrobaeza force-pushed the 14.0-mig-event_registration_cancel_reason branch from c95efb1 to 42bd026 Compare April 22, 2022 08:28
@pedrobaeza pedrobaeza force-pushed the 14.0-mig-event_registration_cancel_reason branch from daede08 to 08f4531 Compare April 22, 2022 08:52
@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-270-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d7811d8 into OCA:14.0 Apr 22, 2022
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 9e345ac. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants