notes_22-06-09.md 1.15 KiB
meeting notes 09.06.22
code review
- python code looks generally quite good
Review 09.6.22
Python:
* jinja2 template satt string concatenation für raw query
* auto_generated_models.py scheint ungültig, wird das nicht verwendet (`db_table = 'spatial_ref_sys# python manage.py inspectdb`?
Quasar:
* pinia -> API-Interaction dorthin verschieben (https://pinia.vuejs.org/core-concepts/actions.html#actions)
* Zu grobgranulare components, Untercomponents sinnvoller
* Stile-Mix (zB imports mit `;` und ohne)
* Magic strings, zB const api = axios.create({ baseURL: 'http://localhost:8000/api/' })
* Das ist für späteres deployen sehr schlecht, da es nie localhost sein wird. Besser: `{ baseURL: '/api/' }`.
Kleinigkeiten
* Schreibfehler: Contributers -> Contributors
other notes
- the code can also be improved after the project is handed in (documentation is fix)
- the abstract should be started very soon -> needs to be final by the 13th
software improvements
-
titel: "OSM Monitoring Tool" in bold
-
trennstrich ueber status buttons noch rausnehmen
-
evtl. noch filtern nach substrings in comment