Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@tupaq
Copy link

@tupaq tupaq commented Feb 19, 2022

No description provided.

@oca-clabot
Copy link

Hey @tupaq, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@legalsylvain legalsylvain added this to the 15.0 milestone Mar 6, 2022
Copy link
Member

@etobella etobella left a comment

Choose a reason for hiding this comment

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

Works as expected

@legalsylvain
Copy link

@bealdav : could you review the migration of the module you shared ?
thanks !

Copy link

@ivantodorovich ivantodorovich left a comment

Choose a reason for hiding this comment

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

Lg

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@ivantodorovich
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 15.0-ocabot-merge-pr-759-by-ivantodorovich-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request May 18, 2022
Signed-off-by ivantodorovich
ivantodorovich and others added 16 commits July 5, 2022 15:56
* Print event.registration receipts.
* Refunding a pos.order will cancel the event registrations.
* Support pricelists.
* Refactored code for better overrideability.
This allows to play nice with other modules, that may either be:

* Trying to modify the created registrations
* Creating registrations themselves
* Or even calling `_create_event_registrations` programatically.

It has no impact on functionality, though.
@tupaq
Copy link
Author

tupaq commented Jul 5, 2022

Ok @ivantodorovich. rebased and fix pre-commit

@ivantodorovich
Copy link

I think there's some issue with your last rebase, because I see some commits for other modules that shouldn't be here.
Could you take care of it? Make sure you pull latests changes from 15.0 before rebasing

image

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@etobella
Copy link
Member

CAn you review the problem? @tupaq

@tupaq
Copy link
Author

tupaq commented Sep 1, 2022

CAn you review the problem? @tupaq

Revised and corrected.

@HaraldPanten
Copy link

@etobella @ivantodorovich could you merge?

@etobella
Copy link
Member

There are some merge commits here... Can you rebase and the push force?

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Feb 19, 2023
@github-actions github-actions bot closed this Mar 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved needs fixing ready to merge stale PR/Issue without recent activity, it'll be soon closed automatically.

Projects

None yet

Development

Successfully merging this pull request may close these issues.