search

LinkopingsStudentspex / internsidor

GNU General Public License v2.0
3 stars 0 forks source link

Rewrite of menu contents #37

Closed Rovanion closed 2 years ago

Rovanion commented 2 years ago

Menyn innehåller nu förhoppningsvis de flesta relevanta länkar en spexare kan tänkas behöva.

Rovanion commented 2 years ago

På min lokala installation så fungerar inte föreställningsräknarlänken. Jag vet inte riktigt hur jag fixar den så du får gärna ta en titt @Karlinde.

Karlinde commented 2 years ago

På min lokala installation så fungerar inte föreställningsräknarlänken. Jag vet inte riktigt hur jag fixar den så du får gärna ta en titt @Karlinde.

Vad är det som inte funkar?

Rovanion commented 2 years ago

Man tryckte på länken i menyn och så fick kom man till ett error snarare än räknaren. Länken i fråga:

https://github.com/LinkopingsStudentspex/internsidor/pull/37/files#diff-56f858131fa163b25c6b105bae4fa23fd6e7417cef313ab1b8500d655ff00a0dR42

Karlinde commented 2 years ago

Man tryckte på länken i menyn och så fick kom man till ett error snarare än räknaren. Länken i fråga:

https://github.com/LinkopingsStudentspex/internsidor/pull/37/files#diff-56f858131fa163b25c6b105bae4fa23fd6e7417cef313ab1b8500d655ff00a0dR42

Hm, det är inget jag ser på min sida. Kan du beskriva närmare vad det är för error du får?

Rovanion commented 2 years ago

RelatedObjectDoesNotExist at /showcounter/

User has no person.

Ser man på. Även är utvecklare är användare när de bestämmer sig för att de inte förstår. Sidan laddas alltså inte för att jag inte har någon person associerad med mitt användarkonto. Men det visas inte ett fint felmeddelande som på inställningssidan utan en stor fet errorsida där det viktiga typ är:

<div id="summary">

Request Method: | GET
-- | --
http://127.0.0.1:8000/showcounter/
3.2.9
RelatedObjectDoesNotExist
User has no person.
/home/rovanion/source/spexet/internsidor/virtual-environment/lib/python3.8/site-packages/django/db/models/fields/related_descriptors.py, line 421, in __get__
/home/rovanion/source/spexet/internsidor/virtual-environment/bin/python
3.8.10
['/home/rovanion/source/spexet/internsidor',  '/usr/lib/python38.zip',  '/usr/lib/python3.8',  '/usr/lib/python3.8/lib-dynload',  '/home/rovanion/source/spexet/internsidor/virtual-environment/lib/python3.8/site-packages']
Wed, 05 Jan 2022 10:30:49 +0000

Så från det här kommer två uppgifter:

  1. [ ] Lägg till ett fint errormeddelande.
  2. [ ] Skriv hur man skapar en person att associera sitt användarkonto med i Readme.md
Karlinde commented 2 years ago

Ah, ja jag trodde jag hade täckt upp det fallet med en ful felsida i showcounter/views.py men det verkar som att jag aldrig testade det på riktigt.

Rovanion commented 2 years ago

Jag gjorde en lite snyggare variant på menyinläggen på botten av skärmen och så "löste" jag problemet med att menyinläggen hamnade utanför skärmen. Lösningen löser inte grundproblemet vilket är att sättet menyn öppnas på gör att dokumentet ökar i höjd såväl som i bredd - men bara på mobiltelefoner. Men grundproblemet orkade jag inte fixa här och nu ikväll, nu är fulhackens tid!