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

[18.0][MIG] mail_activity_done: Migration to 18.0 #17

Open
wants to merge 38 commits into
base: 18.0
Choose a base branch
from

Conversation

achulii
Copy link

@achulii achulii commented Nov 27, 2024

No description provided.

JordiBForgeFlow and others added 30 commits November 27, 2024 14:15
Changed property in test to a correct one

Changed README

Added necessary checks if module does not have an icon

Added default value to _original_module
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/es/
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/it/
Currently translated at 46.6% (7 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/fi/
Currently translated at 60.0% (9 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/fi/
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/fi/
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_done
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_done/fr/
@achulii achulii force-pushed the 18.0-mig-mail_activity_done branch from b368e67 to 5ae6a49 Compare November 27, 2024 16:47
_inherit = "res.users"

@api.model
def _get_activity_groups(self):
Copy link
Member

Choose a reason for hiding this comment

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

this has very little to do with what upstream does by now.

Did you check if we need this override at all? Given this is pure ORM, I'd expect things to just work because done activities are nonactive.

domain=lambda self: [("res_model", "=", self._name), ("active", "=", True)]
)

def read_progress_bar(self, domain, group_by, progress_bar):
Copy link
Member

Choose a reason for hiding this comment

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

given upstream has a test for the active flag in read_group, I doubt we need this override. Did you test this?

@hbrunn
Copy link
Member

hbrunn commented Dec 27, 2024

/ocabot migration mail_activity_done

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Dec 27, 2024
@OCA-git-bot
Copy link
Contributor

There's no issue in this repo with the title 'Migration to version 18.0' and the milestone 18.0, so not possible to add the comment.

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

Successfully merging this pull request may close these issues.