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

[IMP] sale_subscription_views:add graph and pivot view #1073

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from

Conversation

Gerloav
Copy link

@Gerloav Gerloav commented Apr 15, 2024

added graph and pivot view

Copy link
Member

@flotho flotho left a comment

Choose a reason for hiding this comment

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

easy code review.
LGTM
thnaks

@flotho
Copy link
Member

flotho commented Apr 16, 2024

hi @Gerloav , the pipeline has a strange behaviour, any chance to rebase/push something so that it will be trigegred properly ?

@Gerloav Gerloav force-pushed the 16.0-subscription-oca-reports branch from 467de3b to 242f249 Compare April 16, 2024 07:26
@Gerloav
Copy link
Author

Gerloav commented Apr 16, 2024

hi @Gerloav , the pipeline has a strange behaviour, any chance to rebase/push something so that it will be trigegred properly ?

Fixed it! Review please

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

@Gerloav Gerloav force-pushed the 16.0-subscription-oca-reports branch from 4e5491b to 75e6f00 Compare April 29, 2024 11:40
@flotho
Copy link
Member

flotho commented Jul 6, 2024

Hi @Gerloav , any chance to rebase for CI ?

@max3903 max3903 added this to the 16.0 milestone Sep 23, 2024
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.

5 participants