-
-
Notifications
You must be signed in to change notification settings - Fork 407
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
[12.0] [MIG] account_due_list_aging_comment #225
Conversation
Hey @cor9003, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
Hello, The CLA has been already signed and sent by email |
'category': 'Generic Modules/Payment', | ||
'author': 'Eficent,' | ||
'Odoo Community Association (OCA)', | ||
'website': 'http://www.eficent.com', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cor9003 URL https://github.com/OCA/account-payment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use the maintainer-tools to create readme.
@tbaden readme created with maintainer-tools. Thanks to inform me |
Thanks for the work 👍 |
@tbaden The request from @Nikul-Chaudhary is already done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, please Squash your commits into 1
squash commits into 1 done. @tbaden Is there any action I should do to move forward this PR? |
@Nikul-Chaudhary anymore changes or good to go? |
@Nikul-Chaudhary @tbaden Hello, do I have to do anything to move forward this PR ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, maintain commit history as explained here https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-12.0.
To save your work, you can copy the folder and paste outside the repo. Then reset the branch to v12.0, apply what in the link says in order to put add all commit history. And then copy the previous copied folder and paste in the repo overwriting it, and commit and push force.
@mreficent I do not have the branch 12.0 in my forked repository and I don´t know how add this branch 12.0 in my forked repository. I would prefer delete this pull request and start from scratch and give you a clear new pull request. Is it ok for you ? |
ok. do as you prefer. to create branches, you can always do: git fetch origin 12.0 to reset branches, yo only new to do "git reset --hard origin/12" to add remotes, is with "git remote add remote_name |
Migration to 12 account_due_list_aging_comment