Skip to content
Snippets Groups Projects

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