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] add Month and Year in Relative to report base date #19

Merged
merged 1 commit into from
Oct 12, 2017

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Oct 1, 2017

closes #2

@sbidoul sbidoul force-pushed the 9.0-month-year-martronic-sbi branch 2 times, most recently from e51264a to 8f4caef Compare October 1, 2017 13:57
@sbidoul sbidoul changed the base branch from 9.0 to 10.0 October 1, 2017 13:57
@gonzaloruzafa
Copy link

Do you need functional review for this PR? Looks like an interesting feature to add!

@sbidoul
Copy link
Member Author

sbidoul commented Oct 2, 2017

Yes please, review is welcome. Are you in LLN? I'm in the front-left corner of the room.


New features:

* [ADD] month and year reletive periods, easier to use than
Copy link
Member

Choose a reason for hiding this comment

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

typo: relative

@sbidoul sbidoul changed the base branch from 10.0 to 10.0-dev October 3, 2017 15:18
@sbidoul sbidoul added this to the 3.1 milestone Oct 3, 2017
@sbidoul sbidoul force-pushed the 9.0-month-year-martronic-sbi branch from 8f4caef to fe47cb1 Compare October 3, 2017 22:27
@sbidoul sbidoul force-pushed the 9.0-month-year-martronic-sbi branch from fe47cb1 to 28059c8 Compare October 12, 2017 22:16
@sbidoul
Copy link
Member Author

sbidoul commented Oct 12, 2017

Written by @Martronic-SA, review by myself and @lepistone. I merge in 10.0-dev that will become 3.1.

@sbidoul sbidoul merged commit 9430032 into OCA:10.0-dev Oct 12, 2017
@sbidoul sbidoul deleted the 9.0-month-year-martronic-sbi branch October 12, 2017 22:18
@Martronic-SA
Copy link
Contributor

Great! Thank you!

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.

4 participants