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

[17.0][MIG] website_membership_random_order: Migration to 17.0 #187

Merged

Conversation

victoralmau
Copy link
Member

Migration to 17.0

@Tecnativa TT51560

pedrobaeza and others added 6 commits November 15, 2024 16:37
…dom order

Online Members Directory - Random order
=======================================

This modules extends the online members directory for displaying the results
in a random order, and displays a disclaimer about it and the top of the list.

Known issues / Roadmap
----------------------

- All the members will be displayed at once, without pager, as this is the only
  way to get a whole randomized list. On contrary, paging the content, on each
  render, we will get random elements belonging to that slice. Don't use this
  module with a long list of members.
[UPD] Update website_membership_random_order.pot

[BOT] post-merge updates
Currently translated at 100.0% (2 of 2 strings)

Translation: vertical-association-16.0/vertical-association-16.0-website_membership_random_order
Translate-URL: https://translation.odoo-community.org/projects/vertical-association-16-0/vertical-association-16-0-website_membership_random_order/es/
@chienandalu
Copy link
Member

Testing on runbot, I set serveral free members. They show up always in the same order:

image

@victoralmau
Copy link
Member Author

The random mode does not work with free members, only with membership.membership_line records.
This behavior is the same as in previous versions.

@chienandalu
Copy link
Member

Shouldn't it be in the known issues section?

@pedrobaeza
Copy link
Member

/ocabot migration website_membership_random_order

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Nov 18, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 18, 2024
8 tasks
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

OK to add the mention about free members.

@victoralmau victoralmau force-pushed the 17.0-mig-website_membership_random_order branch from a5c1f75 to 76a074f Compare November 18, 2024 08:26
Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

👍

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-187-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit cd65663 into OCA:17.0 Nov 18, 2024
5 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-mig-website_membership_random_order branch November 18, 2024 08:47
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.

8 participants