lnu-norge / lokaler.lnu.no

A website for finding spaces to be in for Norwegian volunteer organizations
http://lokaler.lnu.no/
GNU Affero General Public License v3.0
10 stars 2 forks source link

Space group styling and quality of life improvements #128

Closed DanielJackson-Oslo closed 2 years ago

DanielJackson-Oslo commented 2 years ago

Only lists out SpaceGroup information if there is a SpaceGroup set

So it now looks like this if SpaceGroup is nil:

image

And like this if SpaceGroup is set: image

Changes the space group selection to a dropdown instead of TomSelect

So the user can easily select "nil":

image

Also changes the naming to reflect the move from SpaceOwner to SpaceGroup.

If user has a SpaceGroup it looks like this:

image

Changes the seeds for NSR and dev to import new names

Naming changed from "viken fylkeskommune" to "VGS eid av Viken fylkeskomune"

DanielJackson-Oslo commented 2 years ago

Just found out that TomSelect can do what I want! Making a change back to TomSelect with some changes :)

DanielJackson-Oslo commented 2 years ago

Looks like we are getting an error when selecting the option to add a new space group.

new space group

😬 SÅ mye for å teste happy-path for man sender inn PR! Takk for at du fant, skal fikse.

DanielJackson-Oslo commented 2 years ago

@mathias234 Tror jeg har fikset det nå!

Ikke fått laget en test på det, for vet ikke helt hvordan teste form behaviour. Vet at Ole har snakket om at vi kanskje burde hatt capybara-tester på prosjekter her allikevel, og merker at jeg er litt enig med ham... Hva tror du?

mathias234 commented 2 years ago

@mathias234 Tror jeg har fikset det nå!

Ikke fått laget en test på det, for vet ikke helt hvordan teste form behaviour. Vet at Ole har snakket om at vi kanskje burde hatt capybara-tester på prosjekter her allikevel, og merker at jeg er litt enig med ham... Hva tror du?

Er enig, vi har en veldig tung frontend hvor ting lett kan gå galt. Jeg kan skrive noen I dag så kan vi se hvordan det blir