-
Notifications
You must be signed in to change notification settings - Fork 98
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
Add cost center in account #5967
Conversation
@@ -18,6 +18,7 @@ CREATE TABLE `account` ( | |||
`hidden` TINYINT(1) UNSIGNED DEFAULT 0, | |||
`created` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP, | |||
`reference_id` TINYINT(3) UNSIGNED DEFAULT NULL, | |||
`cost_center_id` MEDIUMINT(8) UNSIGNED NULL, |
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.
You will want to put a FOREIGN KEY here.
@@ -27,6 +27,7 @@ describe('(/accounts) Accounts', () => { | |||
const responseKeys = [ | |||
'id', 'enterprise_id', 'locked', 'created', 'reference_id', | |||
'number', 'label', 'parent', 'type_id', 'type', 'translation_key', | |||
'cost_center_id', |
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.
Can you add a test where you update a cost center to the integration tests? Just for completeness.
@jniles changes landed |
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.
LGTM.
bors r+
Build succeeded: |
After merging IMA-WorldHealth#5967, we can now rewrite these functions to use the account table directly. Closes IMA-WorldHealth#5968.
This PR adds the cost center information in the account.
In the account edit modal:
In the account registry: