Skip to content

Callsign Migration#623

Merged
Syzygicality merged 8 commits intomainfrom
eddie/callsign-migration
Nov 12, 2025
Merged

Callsign Migration#623
Syzygicality merged 8 commits intomainfrom
eddie/callsign-migration

Conversation

@Syzygicality
Copy link
Contributor

@Syzygicality Syzygicality commented Nov 9, 2025

Purpose

Closes #616 .
Explain the purpose of the PR here if it doesn't match the linked issue. Be sure to add a comment in the linked issue explaining the changes.

Check PR for purpose

New Changes

Explain new changes below in short bullet points.

  • added to callsigns.csv (ty julian!) to resources
  • created new model in aro users to store callsigns
  • read csv file with csv lib and return a list of all callsigns
  • checks if callsigns table is already populated. if not, migrate list contents upon app startup
  • automatic B-tree indexing thanks to PostgreSQL

Testing

Explain tests that you ran to verify code functionality.

  • I have unit-tested this PR. Otherwise, explain why it cannot be unit-tested.

I performed startup and callsign data was created in local database (see screenshots below)

  • I have included screenshots of the tests performed below.
  1. delete pre-existing data in callsigns table
image
  1. start app with fastapi dev gs/backend/main.py
image
  1. check db to see newly inserted callsign entries
image

Outstanding Changes

If there are non-critical changes (i.e. additional features) that can be made to this feature in the future, indicate them here.

update wrappers to support new callsigns table.

@Syzygicality Syzygicality self-assigned this Nov 9, 2025
@Syzygicality Syzygicality linked an issue Nov 9, 2025 that may be closed by this pull request
3 tasks
@github-actions
Copy link

github-actions bot commented Nov 9, 2025

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

User Total reviews Time to review Total comments
Adityya-K
🥇
59
▀▀▀▀▀
1d 8h 30m
184
▀▀▀▀
camspec
🥈
29
▀▀▀
1d 8h
172
▀▀▀▀
kepler452b123
🥉
14
6d 7h 42m
37
Yarik-Popov
4
1d 1h 41m
40
proprogrammer504
2
7d 1h 31m
▀▀
7
Navtajh04
1
1d 3h 20m
0
sunray4
1
12d 21m
▀▀▀
9
c4bae
1
13d 1h 35m
▀▀▀
2

⚡️ Pull request stats

@juliansalvador727
Copy link

lgtm. its over 90000!

Copy link
Contributor

@proprogrammer504 proprogrammer504 left a comment

Choose a reason for hiding this comment

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

Please address comments. Though it looks good to me.

Copy link
Contributor Author

@Syzygicality Syzygicality left a comment

Choose a reason for hiding this comment

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

all g

@Syzygicality Syzygicality merged commit a052725 into main Nov 12, 2025
49 checks passed
@Syzygicality Syzygicality deleted the eddie/callsign-migration branch November 12, 2025 03:05
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.

Migrate callsigns to db with indexing

3 participants