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] all: add missing author in all manifests #1044

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

Conversation

Julien00859
Copy link
Member

The author field was made required by the previous commit, this commit add the missing field in all manifests.

The upgrade-code script 18.2-01-manifest-author.py was used. See related community branch (same commit title).

task-4485983

@robodoo
Copy link
Collaborator

robodoo commented Feb 4, 2025

Pull request status dashboard

@Julien00859 Julien00859 force-pushed the master-manifest-mandatory-author-juc branch 2 times, most recently from c64cd08 to 47b1eca Compare February 20, 2025 15:13
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983
@Julien00859 Julien00859 force-pushed the master-manifest-mandatory-author-juc branch from 47b1eca to 7d8963d Compare February 21, 2025 10:25
Copy link

@Levizar Levizar left a comment

Choose a reason for hiding this comment

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

@robodoo
Copy link
Collaborator

robodoo commented Feb 21, 2025

I'm sorry, @Levizar. I'm afraid I can't do that.

@Julien00859
Copy link
Member Author

@robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Feb 21, 2025

@Julien00859 you can't review+.

@d-fence
Copy link
Contributor

d-fence commented Feb 21, 2025

robodoo r+

@robodoo
Copy link
Collaborator

robodoo commented Feb 21, 2025

@Julien00859 @d-fence linked pull request(s) odoo/documentation#11956 not ready. Linked PRs are not staged until all of them are ready.

robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

Part-of: #196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. It has
been removed as part of this commit since it is a one-shot. External
contributors are free to take that script and to adapt it to their
needs.

task-4485983

closes #196408

Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983

closes #1044

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Signed-off-by: Christophe Monniez (moc) <[email protected]>
robodoo pushed a commit to odoo/documentation that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

closes #11956

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/design-themes#1044
Signed-off-by: Julien Castiaux (juc) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

Part-of: #196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. It has
been removed as part of this commit since it is a one-shot. External
contributors are free to take that script and to adapt it to their
needs.

task-4485983

closes #196408

Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983

closes #1044

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Signed-off-by: Christophe Monniez (moc) <[email protected]>
robodoo pushed a commit to odoo/documentation that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

closes #11956

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/design-themes#1044
Signed-off-by: Julien Castiaux (juc) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

Part-of: #196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. It has
been removed as part of this commit since it is a one-shot. External
contributors are free to take that script and to adapt it to their
needs.

task-4485983

closes #196408

Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983

closes #1044

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Signed-off-by: Christophe Monniez (moc) <[email protected]>
robodoo pushed a commit to odoo/documentation that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

closes #11956

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/design-themes#1044
Signed-off-by: Julien Castiaux (juc) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

Part-of: #196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. It has
been removed as part of this commit since it is a one-shot. External
contributors are free to take that script and to adapt it to their
needs.

task-4485983

closes #196408

Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit that referenced this pull request Feb 21, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983

closes #1044

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Signed-off-by: Christophe Monniez (moc) <[email protected]>
robodoo pushed a commit to odoo/documentation that referenced this pull request Feb 21, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

closes #11956

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/design-themes#1044
Signed-off-by: Julien Castiaux (juc) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 22, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

Part-of: #196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit to odoo/odoo that referenced this pull request Feb 22, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. It has
been removed as part of this commit since it is a one-shot. External
contributors are free to take that script and to adapt it to their
needs.

task-4485983

closes #196408

Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Related: odoo/design-themes#1044
Signed-off-by: Brice Bartoletti (bib) <[email protected]>
robodoo pushed a commit that referenced this pull request Feb 22, 2025
The author field was made required by the previous commit, this commit
add the missing field in all manifests.

The upgrade-code script `18.2-01-manifest-author.py` was used. See
related community branch (same commit title).

task-4485983

closes #1044

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/documentation#11956
Signed-off-by: Christophe Monniez (moc) <[email protected]>
robodoo pushed a commit to odoo/documentation that referenced this pull request Feb 22, 2025
External contributors (and namely Odoo PS) forget to set the author
field in their manifest file. The result is that the module is wrongly
registered with `'author': 'Odoo S.A.'`.

This work makes the field mandatory, with a warning when not set. It
also offers an upgrade-code script to mass-rewrite all the manifests in
order to set the author.

task-4485983

closes #11956

Related: odoo/odoo#196408
Related: odoo/enterprise#78513
Related: odoo/design-themes#1044
Signed-off-by: Julien Castiaux (juc) <[email protected]>
@robodoo
Copy link
Collaborator

robodoo commented Feb 22, 2025

@Julien00859 @d-fence linked pull request(s) odoo/enterprise#78513 not ready. Linked PRs are not staged until all of them are ready.

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