Skip to content

Add UKCP positions syncing#4965

Open
kristiankunc wants to merge 4 commits into
mainfrom
tech-649-sync-ukcp-positions-to-core
Open

Add UKCP positions syncing#4965
kristiankunc wants to merge 4 commits into
mainfrom
tech-649-sync-ukcp-positions-to-core

Conversation

@kristiankunc

@kristiankunc kristiankunc commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Waiting for BW to review UKCP positions

Adds a scheduled sync (daily at 04:00) that pulls controller positions from the UKCP API into the core positions table. New positions are created, existing ones have frequency/ID updated, and positions removed from UKCP are soft-deleted. Includes ukcp_position_id column, soft deletes on the model, a helper to infer position type from callsign, and removes the unused sub_station column. Supports --dry-run for preview.

@linear

linear Bot commented Jul 1, 2026

Copy link
Copy Markdown

TECH-649

@kristiankunc kristiankunc requested a review from CLC0609 July 1, 2026 19:54

@CLC0609 CLC0609 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Other than that all looks good!

Comment thread app/Filament/Admin/Resources/Positions/PositionResource.php
Comment on lines +140 to +141
'APP', 'APPROACH', 'F_APP', 'N_APP', 'S_APP', 'A_APP',
'L_APP', 'D_APP', 'R_APP', 'P_APP', 'Z_APP' => self::TYPE_APPROACH,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missed this on my review, but we probably don't need all the *_APP callsigns as the split would only grab the last part anyways right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants