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

[BRMO-380] Oracle view vb_util_zk_recht_op_koz toont niet alle zakelijk rechten #2283

Merged
merged 2 commits into from
Nov 1, 2024

Conversation

KJLammers
Copy link
Contributor

Aanpassingen aan Oracle view ihkv BRMO-380

@KJLammers KJLammers requested a review from mprins October 22, 2024 11:17
@mprins mprins changed the title Brmo 380 [BRMO-380] Oracle view vb_util_zk_recht_op_koz toont niet alle zakelijk rechten Nov 1, 2024
Copy link
Member

@mprins mprins left a comment

Choose a reason for hiding this comment

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

titel van PR aangepast zodat deze op de juiste manier in de release notes komt

N.a.v. support vraag geconstateerd dat in de Oracle versie van deze view op regel 508 gekeken werd naar een verkeerde tabel. Hierdoor waren alleen eigendomsrechten aan kadastrale onroerende zaken gekoppeld. Hiermee wordt dit probleem gefixed
@mprins mprins enabled auto-merge (squash) November 1, 2024 11:08
Copy link

codecov bot commented Nov 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38%. Comparing base (bc5aa92) to head (c1aa99d).
Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             master   #2283   +/-   ##
========================================
- Coverage        38%     38%   -0%     
  Complexity      806     806           
========================================
  Files           371     371           
  Lines         18863   18863           
  Branches       1777    1777           
========================================
- Hits           7153    7152    -1     
  Misses        11138   11138           
- Partials        572     573    +1     

see 1 file with indirect coverage changes

@mprins mprins merged commit 08d7e1e into master Nov 1, 2024
6 checks passed
@mprins mprins deleted the BRMO-380 branch November 1, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants