Skip to content

34088 Colin API - business snapshot endpoint - #4711

Merged
kialj876 merged 4 commits into
bcgov:mainfrom
kialj876:34088-colin-api-snapshot-endpoint
Aug 20, 2026
Merged

34088 Colin API - business snapshot endpoint#4711
kialj876 merged 4 commits into
bcgov:mainfrom
kialj876:34088-colin-api-snapshot-endpoint

Conversation

@kialj876

Copy link
Copy Markdown
Collaborator

Issue #: /bcgov/entity#34088

Description of changes:

  • added colin business snapshot which will be used for pulling all the details we need for amalgamation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the lear license (Apache 2.0).

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@kialj876 kialj876 self-assigned this Aug 19, 2026
Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@codecov

codecov Bot commented Aug 19, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.00000% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.12%. Comparing base (d995156) to head (4d93bd1).
⚠️ Report is 656 commits behind head on main.

Files with missing lines Patch % Lines
...olin-api/src/colin_api/models/business_snapshot.py 91.13% 7 Missing ⚠️
colin-api/src/colin_api/models/filing.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4711      +/-   ##
==========================================
+ Coverage   29.66%   33.12%   +3.46%     
==========================================
  Files          39       40       +1     
  Lines        3486     3704     +218     
==========================================
+ Hits         1034     1227     +193     
- Misses       2452     2477      +25     
Flag Coverage Δ
colinapi 33.12% <92.00%> (+3.46%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
colin-api/src/colin_api/models/__init__.py 100.00% <100.00%> (ø)
colin-api/src/colin_api/models/shares.py 49.68% <100.00%> (+21.02%) ⬆️
colin-api/src/colin_api/resources/business.py 50.00% <100.00%> (+12.93%) ⬆️
colin-api/src/colin_api/version.py 100.00% <100.00%> (ø)
colin-api/src/colin_api/models/filing.py 13.20% <0.00%> (-0.03%) ⬇️
...olin-api/src/colin_api/models/business_snapshot.py 91.13% <91.13%> (ø)

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Kial Jinnah <kialj876@gmail.com>
Signed-off-by: Kial Jinnah <kialj876@gmail.com>
@sonarqubecloud

Copy link
Copy Markdown

@kialj876
kialj876 merged commit 8cca516 into bcgov:main Aug 20, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants