mrawls
commented
3 years ago Excellent work, thank you for the PR!! Initial comments from discussion:
- Dino will do a formal review
- Consider using django pagination, or ensure your custom pagination handles edge cases by adding tests
- Add docstrings to all python functions, you can follow this guide for best practices
- Will likely want to move the
imageandget_imageurls into the mainrender_gallerycall
Things that this pull request contains:
Import images and their metadata to the gallery page to be displayed. When clicking an image it redirects to a page for that image with the metadata being displayed. Adjusted table so that there is less white space. Added an asynchronous request system for getting new images and loading them onto the main webpage.
Things that probably need to be worked on:
Obtaining data from the database seems inefficient, could probably be better. Downloading images.