Skip to content

[ADD] vcp_psc - #15

Open
etobella wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-pax
Open

etobella wants to merge 1 commit into
OCA:18.0from
dixmit:18.0-pax

Conversation

@etobella

Copy link
Copy Markdown
Member

@sebastienbeau It might have sense to have this in the OCA Repository.

Also, we could show the information in the Groups of the users. WDYT?

I also have the one that allows to know how many translations were done by each one.

Might be interesting too, isn't it?

@TumbaoJu

@TumbaoJu

Copy link
Copy Markdown

@sebastienbeau It might have sense to have this in the OCA Repository.

Also, we could show the information in the Groups of the users. WDYT?

I also have the one that allows to know how many translations were done by each one.

Might be interesting too, isn't it?

@TumbaoJu

Yes it could be interesting.

from odoo import fields, models


class VCPPlatform(models.Model):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Small convention, in the existing module we have use CamelCase

Suggested change
class VCPPlatform(models.Model):
class VcpPlatform(models.Model):

from odoo import fields, models


class VCPPlatformPsc(models.Model):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
class VCPPlatformPsc(models.Model):
class VcpPlatformPsc(models.Model):

from odoo import fields, models


class VCPRepository(models.Model):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
class VCPRepository(models.Model):
class VcpRepository(models.Model):

from odoo import fields, models


class VCPUser(models.Model):

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
class VCPUser(models.Model):
class VcpUser(models.Model):

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants