-
Notifications
You must be signed in to change notification settings - Fork 0
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][IMP] base_edi: introduce a basic edi user group #33
base: 12.0
Are you sure you want to change the base?
Conversation
5225790
to
e9f4ba7
Compare
e9f4ba7
to
42acacc
Compare
@@ -6,7 +6,7 @@ | |||
"name": "Base EDI", | |||
"summary": """Base module to aggregate EDI features.""", | |||
"version": "12.0.1.0.1", | |||
"development_status": "Alpha", | |||
"development_status": "Beta", |
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.
what motivates this change?
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.
The edi_oca module has a development_status of "beta" and depends on a module with a development_status of "alpha" and got a warning in test. And in base_edi v14 there is a commit to move it (OCA@e5db4e0)
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.
It's so little that I'll fix it quickly. Or should I cherry-pick it?
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.
keep it as is, but make it explicit in PR description why you had to do this
Context
Changes