Skip to content

[FIX] - handle missing lat/long on UNTP credential publish #3702

Merged
Jsyro merged 20 commits intodevelopfrom
feature/regional-untp-map
Oct 15, 2025
Merged

[FIX] - handle missing lat/long on UNTP credential publish #3702
Jsyro merged 20 commits intodevelopfrom
feature/regional-untp-map

Conversation

@Jsyro
Copy link
Collaborator

@Jsyro Jsyro commented Oct 15, 2025

Found some regional mines without lat/long.

Do not publish UNTP creds for them.

image (7)

@Jsyro Jsyro added 💾 Backend This pull request includes backend changes. 💊 Fix Fixes something that isn't working :) labels Oct 15, 2025
matbusby-fw
matbusby-fw previously approved these changes Oct 15, 2025
Copy link
Collaborator

@matbusby-fw matbusby-fw left a comment

Choose a reason for hiding this comment

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

👍🏼

@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-api'

Failed conditions
33.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

@Jsyro Jsyro merged commit a3b5eec into develop Oct 15, 2025
12 of 13 checks passed
@Jsyro Jsyro deleted the feature/regional-untp-map branch October 15, 2025 19:11
matbusby-fw pushed a commit that referenced this pull request Nov 5, 2025
* param fix, mine_Type null check

* add regional para to job

'flask push_untp_map_data_to_publisher True' to use

* comment WIP line.... oops

* endpoint and path to test endpoint

* sort product names to avoid duplicates.

* new testing endpoint

* fixed the testing endpoint

* comment

* remove log

* try both methods to try

* run the individual job and compare

* method was already in this file.

* remove import

* safety if lat or lon is missing

* don't scope location_information string

* update log string

---------

Signed-off-by: Jason Syrotuck <jasyrotuck@gmail.com>
Co-authored-by: Jason Syrotuck <177293@nttdata.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

💾 Backend This pull request includes backend changes. 💊 Fix Fixes something that isn't working :)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants