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.

Sign up for GitHub

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

Jump to bottom

#6451 Adjust and corect sums for tax accounts report #6498

Merged
merged 21 commits into from Apr 16, 2020
Merged

#6451 Adjust and corect sums for tax accounts report #6498

merged 21 commits into from Apr 16, 2020

Conversation

cristinamghita
Copy link
Member

  • Update old report to use DateForm - DateTo as parameters instead of period
  • Create a new report that shows the correct amounts, as well the year balance and period balance
  • Create a materialized view for speeding up retrieving data; The MV is refreshed with a scheduler, each night

@cristinamghita
#6451 Add dateFrom and DateTo as params instead of C_Period_ID
632afb5
@cristinamghita
#6451 Add indexes
f8e80fd 
* add indexes on fact_acct to omprove perfomance
* both indexes: query runs in 5 s 514 ms
* no index: query runs in 7 m 30 s 305 ms
* old query from report.tax_accounting_report was running in 3 m 17 s 565 ms
@cristinamghita
Merge branch 'master' into gh6451
07883d4
@cristinamghita
#6451 Add funciton for getting the balance
36e3ebd
@cristinamghita
#6451 Adjust tax__accounting_report funciton to match the new logic
45bf3e0
@cristinamghita
#6451 Show coorect data, summing the correct taxes
18378e1
@cristinamghita
#6451 Show coorect data, summing up the correct taxes WIP
094f744
@cristinamghita
Merge branch 'master' into gh6451
4709282
@cristinamghita
#6451 Show balance to year and current period
1f44e1f
@cristinamghita
#6451 Fix all sums
59957e3
@cristinamghita
#6451 Speed up the function
fbe7bb6
@cristinamghita
#6451 Add materialized view and functions to speed up reporting
65a021e
@cristinamghita
Merge branch 'master' into gh6451
6d9573d
@cristinamghita
#6451 Add scheduler to refresh the MV each night
e959dd0
@cristinamghita
#6451 Leave the old repor as it is
c359891
@cristinamghita
#6451 Add new separate report
d65bbb5
@cristinamghita
#6451 Set proper path
d21bce8
@cristinamghita
#6451 Add new report
00a39cd
@cristinamghita
#6451 Add funciton and adjust reports
58f868c
@cristinamghita
#6451 Replace parameters (period with DateForm - DateTo)
e5cde5b
@cristinamghita
#6498 Add tax_accounting_report_details function
295e5f7
@metas-ts metas-ts merged commit 14db17a into master Apr 16, 2020
@metas-ts metas-ts deleted the gh6451 branch April 16, 2020 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@metas-ts metas-ts metas-ts approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@cristinamghita @metas-ts