Skip to content

32886 - SAF - add ability to filter businesses based on director mail… #4254

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

Merged
Rajandeep98 merged 3 commits into from
Apr 14, 2026
Merged

32886 - SAF - add ability to filter businesses based on director mail… #4254

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9f918f5
32886 - SAF - add ability to filter businesses based on director mail…
Rajandeep98 Apr 9, 2026
1c0b320
Merge branch 'bcgov:main' into 32886_Filter_Directors_On_Address
Rajandeep98 Apr 14, 2026
ec26ae4
updated
Rajandeep98 Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions data-tool/scripts/colin_corps_extract_postgres_views_ddl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -842,6 +842,8 @@ SELECT COALESCE(edg.group_name, NULL) AS group_name,
CAST(recognition_dts as date) as recognition_dts,
CAST(last_ar_filed_dt as date) as last_ar_filed_dt,
CAST(director_count as integer) as director_count,
CAST(directors_within_bc as boolean) as directors_within_bc,
CAST(directors_within_ca as boolean) as directors_within_ca,
CAST(filing_cnt as integer) as filing_cnt,
CAST(months_since_last_ar_filing as integer) as months_since_last_ar_filing,
CAST(ar_unfiled_over_1yr as boolean) as ar_unfiled_over_1yr,
Expand Down Expand Up @@ -920,6 +922,18 @@ FROM (
ev.event_file_types,
ev.failed_events,
COALESCE(cprt_dir.director_count, 0) AS director_count,
CASE
WHEN COALESCE(dir_mail_loc.dir_cnt, 0) > 0
AND COALESCE(dir_mail_loc.dir_bc_mailing_cnt, 0) = dir_mail_loc.dir_cnt
THEN 'Y'
ELSE 'N'
END AS directors_within_bc,
CASE
WHEN COALESCE(dir_mail_loc.dir_cnt, 0) > 0
AND COALESCE(dir_mail_loc.dir_ca_mailing_cnt, 0) = dir_mail_loc.dir_cnt
THEN 'Y'
ELSE 'N'
END AS directors_within_ca,
CASE WHEN cow.corp_num IS NOT NULL THEN 'Y' ELSE 'N' END AS has_officers,
CASE WHEN cowtp.corp_num IS NOT NULL THEN 'Y' ELSE 'N' END AS has_3rd_party,
CASE WHEN cowtp.corp_num IS NOT NULL THEN cowtp.vendor ELSE '' END AS vendor,
Expand Down Expand Up @@ -974,6 +988,26 @@ FROM (
WHERE party_typ_cd = 'DIR'
GROUP BY corp_num
) cprt_dir ON cprt_dir.corp_num = c.corp_num
LEFT OUTER JOIN (
SELECT
cp.corp_num,
COUNT(*) AS dir_cnt,
COUNT(*) Filter (
WHERE cp.mailing_addr_id IS NOT NULL
Copy link
Copy Markdown
Collaborator

@argush3 argush3 Apr 10, 2026

Choose a reason for hiding this comment

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

think we should filter out stub addresses too. believe the stub address always has an address id = 1 in the COLIN extract.

i.e. add AND cp.mailing_addr_id <> 1 to where clause

AND cp.mailing_addr_id <> 1
AND UPPER(TRIM(ma.province)) = 'BC'
) AS dir_bc_mailing_cnt,
COUNT(*) Filter (
WHERE cp.mailing_addr_id IS NOT NULL
AND cp.mailing_addr_id <> 1
AND UPPER(TRIM(ma.country_typ_cd)) = 'CA'
) AS dir_ca_mailing_cnt
FROM corp_party cp
LEFT JOIN address ma ON ma.addr_id = cp.mailing_addr_id
WHERE cp.end_event_id is NULL
AND cp.party_typ_cd = 'DIR'
GROUP BY cp.corp_num
) dir_mail_loc ON dir_mail_loc.corp_num = c.corp_num
LEFT OUTER JOIN mv_admin_email_count aec ON c.admin_email = aec.admin_email
LEFT OUTER JOIN mv_addr_quality_screening_by_corp mvaqs ON c.corp_num = mvaqs.corp_num
LEFT OUTER JOIN bar_corps bc ON c.corp_num = bc.identifier
Expand Down