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

[9.0] information_security_manual #97

Merged
merged 3 commits into from
Mar 6, 2016

Conversation

naousse
Copy link
Contributor

@naousse naousse commented Oct 30, 2015

No description provided.

Information Security Management System Manual
=============================================

This module was written to provide an Information Security Manual Template.

Choose a reason for hiding this comment

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

This module provides an Information Security Manual Template.

@max3903 max3903 changed the title Migration of information_security_manual to Odoo 9.0 [9.0] information_security_manual Nov 3, 2015
@max3903 max3903 added this to the 9.0 milestone Nov 3, 2015
@pedrobaeza pedrobaeza mentioned this pull request Nov 3, 2015
30 tasks
@naousse naousse force-pushed the 9.0-information_security_manual branch from 7b1c597 to b9130c9 Compare January 29, 2016 23:34
@max3903
Copy link
Sponsor Member

max3903 commented Feb 4, 2016

👍

Contributors
------------

* Odoo SA <info@odoo.com>
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

Really?

@dreispt
Copy link
Sponsor Member

dreispt commented Feb 8, 2016

1/ The module named should be prefixed with mgmtsystem_
2/ The module just adds a record on a model? Is that it?

@max3903
Copy link
Sponsor Member

max3903 commented Feb 8, 2016

@dreispt
1/ If we add the prefix, the name is tool long and truncated.
2/ The module adds the template of a ISMS manual based on ISO27001

@dreispt
Copy link
Sponsor Member

dreispt commented Feb 10, 2016

You could use a shorter name; mgmtsystem_infosec_manual?
Unless I'm missing something, I see added only a documennt with a <h1>Application Domain</h1> title.

@max3903
Copy link
Sponsor Member

max3903 commented Feb 10, 2016

@dreispt I think infosec is bad for SEO and yes, you missed the content of the category which hasn't changed:
https://github.com/naousse/management-system/blob/9.0-information_security_manual/information_security_manual/data/information_security_manual.xml

@dreispt
Copy link
Sponsor Member

dreispt commented Feb 11, 2016

I still still we should have the module area prefix. I'm sure you can come up with a shorter module name.

@max3903
Copy link
Sponsor Member

max3903 commented Feb 11, 2016

@dreispt Honestly, I don't think of any better one than yours...

@naousse Can you rename the module with Daniel's suggestion please?

@max3903 max3903 self-assigned this Feb 11, 2016
@dreispt
Copy link
Sponsor Member

dreispt commented Feb 12, 2016

@max3903 Since "infosec" is quite a specific keyword, what about mgmtsystem_info_sec_manual?

Maxime Chambreuil and others added 2 commits February 12, 2016 11:17
@max3903
Copy link
Sponsor Member

max3903 commented Mar 4, 2016

@dreispt ping!

@dreispt
Copy link
Sponsor Member

dreispt commented Mar 5, 2016

👍

max3903 pushed a commit that referenced this pull request Mar 6, 2016
@max3903 max3903 merged commit 5b5db9e into OCA:9.0 Mar 6, 2016
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.

None yet

5 participants