Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Incorrect MIS report p/l #72

Closed
ThijsvOers opened this issue Oct 4, 2023 · 6 comments
Closed

Incorrect MIS report p/l #72

ThijsvOers opened this issue Oct 4, 2023 · 6 comments

Comments

@ThijsvOers
Copy link
Contributor

The following accounts are missing in the MIS report

8011100 | Opbrengsten servicecontracten
8012100 | Lasten servicecontracten
8014100 | Lasten verhuur en beheeractiviteiten
8015100 | Lasten onderhoudsactiviteiten
8016100 | Overige directe operationele lasten explotatie bezit
8030100 | Verkoopopbrengst vastgoedportefeuille
8032100 | Boekwaarde verkochte vastgoedportefeuille
8033100 | Verkoopkosten
8040100 | Overige waardeveranderingen van vastgoedportefeuille
8050100 | Opbrengsten overige activiteiten
8051100 | Kosten overige activiteiten

@ThijsvOers
Copy link
Contributor Author

@astirpe yes in the MIS P/L

@astirpe
Copy link
Contributor

astirpe commented Oct 5, 2023

@ThijsvOers shall those accounts appear under the "Netto-omzet" section of the MIS P/L?

Here is the mapping with the groups:

Account Account Name Group KPI Name Value type Expression
8011100 Opbrengsten servicecontracten 8011000 nl_80_11 Numeric -balp[('group_id.code', '=', '8011000')]
8012100 Lasten servicecontracten 8012000 nl_80_12 Numeric -balp[('group_id.code', '=', '8012000')]
8014100 Lasten verhuur en beheeractiviteiten 8014000 nl_80_14 Numeric -balp[('group_id.code', '=', '8014000')]
8015100 Lasten onderhoudsactiviteiten 8015000 nl_80_15 Numeric -balp[('group_id.code', '=', '8015000')]
8016100 Overige directe operationele lasten explotatie bezit 8016000 nl_80_16 Numeric -balp[('group_id.code', '=', '8016000')]
8030100 Verkoopopbrengst vastgoedportefeuille 8030000 nl_80_30 Numeric -balp[('group_id.code', '=', '8030000')]
8032100 Boekwaarde verkochte vastgoedportefeuille 8032000 nl_80_32 Numeric -balp[('group_id.code', '=', '8032000')]
8033100 Verkoopkosten 8033000 nl_80_33 Numeric -balp[('group_id.code', '=', '8033000')]
8040100 Overige waardeveranderingen van vastgoedportefeuille 8040000 nl_80_40 Numeric -balp[('group_id.code', '=', '8040000')]
8050100 Opbrengsten overige activiteiten 8050000 nl_80_50 Numeric -balp[('group_id.code', '=', '8050000')]
8051100 Kosten overige activiteiten 8051000 nl_80_51 Numeric -balp[('group_id.code', '=', '8051000')]

CC @yung-wang

@astirpe
Copy link
Contributor

astirpe commented Oct 5, 2023

Is this ok or should be grouped in a different way?

Screenshot from 2023-10-05 10-12-20

Changes tracked here: #73

@astirpe astirpe removed their assignment Oct 5, 2023
@ThijsvOers ThijsvOers assigned astirpe and unassigned ThijsvOers and astirpe Oct 10, 2023
@ThijsvOers
Copy link
Contributor Author

@astirpe how can I test this change? is the change applied to the current Curq databases? TIA

@astirpe
Copy link
Contributor

astirpe commented Oct 10, 2023

@ThijsvOers I merged the PR and start the jenkins. You will be able to test it in 30 min

@ThijsvOers
Copy link
Contributor Author

Tested with an import of all accounts, and all accounts are in the P/L

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

No branches or pull requests

2 participants