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

[MIG] account_cutoff_start_end_dates: Migration to 16.0 #216

Merged
merged 13 commits into from
Feb 16, 2023

Conversation

dzungtran89
Copy link

@dzungtran89 dzungtran89 mentioned this pull request Dec 7, 2022
7 tasks
@dzungtran89 dzungtran89 force-pushed the 16.0-account_cutoff_start_end_dates branch 3 times, most recently from 6cffee5 to 0d575a5 Compare January 5, 2023 09:28
@dzungtran89 dzungtran89 force-pushed the 16.0-account_cutoff_start_end_dates branch from 0d575a5 to 32a1d58 Compare January 18, 2023 04:16
@alexis-via
Copy link
Contributor

I made a PR to improve this PR:
dzungtran89#2

@yankinmax
Copy link

@dzungtran89 can you pls rebase to fix the build?

@dzungtran89 dzungtran89 force-pushed the 16.0-account_cutoff_start_end_dates branch from e2b356b to f71ee61 Compare February 16, 2023 09:04
To simplify the code and the user interface, the forecast mode doesn't
use a boolean field any more; it is now a specific "state".
Migration script is provided.
@dzungtran89 dzungtran89 force-pushed the 16.0-account_cutoff_start_end_dates branch from f71ee61 to a0aca44 Compare February 16, 2023 09:11
@dzungtran89
Copy link
Author

@yankinmax the PR has been updated, could you please check?

@alexis-via
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 16.0-ocabot-merge-pr-216-by-alexis-via-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit df0139c into OCA:16.0 Feb 16, 2023
@OCA-git-bot
Copy link
Contributor

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

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.

7 participants