OCA / event

Event Management addons
GNU Affero General Public License v3.0
62 stars 166 forks source link

[17.0][MIG] website_event_membership_restriction: Migration to version 17.0 #411

Closed pilarvargas-tecnativa closed 1 week ago

pilarvargas-tecnativa commented 2 weeks ago

cc @Tecnativa TT51547

@chienandalu @carlos-lopez-tecnativa please review

pedrobaeza commented 2 weeks ago

/ocabot migration website_event_membership_restriction

pilarvargas-tecnativa commented 2 weeks ago

I'm testing the functionality in runboat and it doesn't seem to work. I set up the membership of the portal user but I still can't register in the restricted event...

I have tested it and it works correctly, maybe due to the validity date of the membership it was not active?

image

image

chienandalu commented 2 weeks ago

I could make it work, but now I cancelled the subscription but I can still register...

image

pilarvargas-tecnativa commented 2 weeks ago

I could make it work, but now I cancelled the subscription but I can still register...

Maybe it's a runboat problem?

image

pilarvargas-tecnativa commented 1 week ago

Try two different users in runbot, DEMO has paid membership and ADMIN does not: image

image

I now cancel the DEMO membership: image

This problem arises in runbot, it is not updating the user's membership status. To update it you have to log out and log in again but this problem does not arise in local that the update of the membership status is correct without having to log out. In any case, the problem is not related to the module, it is a runbot issue.

I will leave in the template the data taken by screen in case you want to do more tests

pedrobaeza commented 1 week ago

But this is something outside of the scope of the module AFAIK.

pilarvargas-tecnativa commented 1 week ago

Tested and confirmed working for me. Can this content be removed? That content is in the runbot view, when the runbot is reset it will be deleted.

OCA-git-bot commented 1 week ago

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). 🤖

pedrobaeza commented 1 week ago

/ocabot merge nobump

OCA-git-bot commented 1 week ago

On my way to merge this fine PR! Prepared branch 17.0-ocabot-merge-pr-411-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 1 week ago

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