amidaware / tacticalrmm

A remote monitoring & management tool, built with Django, Vue and Go.
https://docs.tacticalrmm.com
Other
2.92k stars 422 forks source link

initial SysTray api PR #1829

Open conlan0 opened 2 months ago

conlan0 commented 2 months ago
conlan0 commented 2 months ago

do i need to add my migrations here too?

wh1te909 commented 2 months ago

yes migrations need to be added

conlan0 commented 2 months ago

how do i fix these errors

wh1te909 commented 2 months ago

look at the test failures to see what it's complaining about, in this case you need to format your code with black

The systray should be it's own django app, not under apiv3 so please move it out of there. You can still write the views under apiv3/views.py. Also makes more sense to name the app systray instead of support. Run python manage.py startapp systray under the project root. You'll need to delete and recreate your migrations and adjust imports etc.

conlan0 commented 2 months ago

Okay awesome thanks, will take care of those and get it updated!