Clayton-TV / claytontv

3 stars 1 forks source link

Add views for the pages that we know we are going to need, to streamline front end development #34

Open Fehings opened 5 months ago

Fehings commented 5 months ago

Django requires 'views' to exist for it to recognise webpages: image

Ettie has made a site map overview which we can use to pre-populate the views with pages we know we are going to need. https://lucid.app/lucidspark/a6b575a8-0ab8-403e-ae35-3a9b747f720e/edit?viewport_loc=-2199%2C-389%2C6650%2C3183%2C0_0&invitationId=inv_a96a9499-794a-42b4-a39d-f0355c9a35ba

that's it really, this should be a quick little one as a nice starter! For a guide on what these views should look like see:

darthpedroo commented 5 months ago

hey! I wanted to contribute but I had a question. You have 2 different apps, which are "catalogue" and "livestreams". In the site map overview, you show different routes and I don't know in which app they should be or if you want to create a different app for each "view/tag". image For example, I understand that "About" should be a route with its corresponding view, however, I don't know whether you want to add it to the livestreams or catalogue app or if you would like to create a new app. If you could answer this I would really love to help and contribute to the project since it would be one of my first contributions to open source :)

Fehings commented 5 months ago

@darthpedroo good question. The current plan is that the majority of the site will have views in 'catalogue'. The livestream app is only for pages specifically related to livestreams (to keep that part of the site conceptually separate, as we will want that to be more minimalist - the main target audience for livestreams is different, namely elderly tech illiterate folks who need to to be as straightforward as possible). So in terms of the site map, only 'livestreams' and the livestream archive subset will have views in the livestream app and everything else should have a view in the catalogue app. Does that make sense? Happy to take feed back on this if you have thoughts on how to do it better!

Fehings commented 4 months ago

I believe this has been done, part of the staff page pull request which needs sorting before this can be closed.