-
Notifications
You must be signed in to change notification settings - Fork 151
added mentions legales to technical settings / tenant back #3898
base: master-qa
Are you sure you want to change the base?
Conversation
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.
My 2 cents
Like the old time :-)
}, | ||
"legalStatus": { | ||
"type": "string", | ||
"sanitize": "mongo" | ||
} |
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.
Maybe all this data goes to common.js as it can be reused in Company ;P
And why not making it a component like the address ?
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 could be reused by companies but we won't use it as seen with @ClaudeROSSI
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.
and yes i should have made a new component as address or support 🤷🏻♀️
@@ -20,6 +20,34 @@ | |||
}, | |||
"components": { | |||
"$ref": "tenant-components#/definitions/components" | |||
}, | |||
"support": { |
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.
Also generic, can be reused in Company for instance ;-)
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.
same comment
No description provided.