AlbertaSat / ex2_ground_station_website

Ground station user interface for Ex-Alta 2.
6 stars 3 forks source link

Import commands in manage.py from ground_station_software instead of hardcoding #60

Closed castorway closed 2 years ago

jmmabanta commented 2 years ago

@ahmedsiad If you have the time, can you make sure that automated commands on passovers work as expected with these new command prefix changes? It seems to work fine on my end but you may find something broken that I might've missed

ahmedsiad commented 2 years ago

@jmmabanta Everything looks good on my end. Only thing I've noticed is that when you select a command to add in the automated command list, all other commands with a different prefix but the same command also get highlighted since they all share the same value. You could fix it by making all commands increment by 1 in value and to find the actual command id just do integer division with the number of prefixes. It isn't too much of an issue though so its not too important.

jmmabanta commented 2 years ago

@ahmedsiad Thanks for the suggestion! In the end, I decided to just add a copy of each command prefixed with each server to the database as it made things much easier (though seeding the database is a bit slower now). It also fixed the highlighting bug you've mentioned!