Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/admin documents #12

Merged
merged 7 commits into from
Mar 30, 2023
Merged

Feature/admin documents #12

merged 7 commits into from
Mar 30, 2023

Conversation

Kremtastic
Copy link
Contributor

Add "public documents" page with functional links.

Copy link
Member

@timharek timharek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Godt jobba! Veldig kult at du har fått til å lage subsections med undersider 😄

Jeg hadde noen kommentarer som ligger i koden. Anbefaler å ta en titt på dokumentasjonen til Zola når du jobber med templates og ikke bare Markdown ☺️ Også er det bare å spørre om hjelp enten her eller via Mattermost

@Kremtastic
Copy link
Contributor Author

Endringer er blitt gjort.

@Cengelsen
Copy link
Member

Cengelsen commented Oct 14, 2022

Ser veldig bra ut! Alt som gjenstår er å legge inn alle dokumentene for handlingsplaner, møtereferater og rapporter om organisasjonens virksomhet, samt resten av dagsordene. Jeg tenker Konstituering og Statutter kan lenke til en egen side 😄

Copy link
Member

@timharek timharek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dritbra jobba! 🙌

Mangler PRen flere dokumenttyper? Jeg kan kun se at det er lagt til dagsordner, men ikke for de andre.

Hadde også et par kommentarer for litt småting som whitespace og litt annet knask.

@timharek
Copy link
Member

@Cengelsen hvis/når denne PRen blir merget, betyr det at alle dokumenter må skrives dobbelt opp? At vi må legge dem til i admin-repoet for også legge til nøyaktig samme fil i fribyte.no-repoet? 💭

@Cengelsen
Copy link
Member

Cengelsen commented Oct 14, 2022

Det ender opp med at det blir sånn ja. Om det finnes en måte å importere dem fra et lukket repo, så hadde jo det vært bedre

@mathiash98 mathiash98 requested review from timharek and Cengelsen and removed request for timharek February 21, 2023 17:06
@timharek
Copy link
Member

Zola v.0.17.0 har en bug, getzola/zola#2101, som gjør at jeg ikke får testet lokalt. Men hvis det funker for dere andre så ser jeg ikke noe i veien med å merge hvis dere venter på meg 😅

Copy link
Member

@timharek timharek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 🙌

Mange filer å se gjennom, har lagt til noen kommentarer. Ser også at det er litt filer som har to frontmatter, med TOML (+++) og YAML (---), feks dagsorden-2022-11-24.md har dette, det blir ikke bygget riktig.

Også mangler det fremdeles dokumenter for:

  • Handlingsplaner
  • Konstituering
  • Møtereferater
  • Rapport
  • Statutter

Ser date blir brukt i filene, det gjør at RSS-feeden vår vil bli oppdatert med alle dokumentene som er datert, se dokumentasjonen. Dette kan løses med å bruke feks.:

+++
[extra]
date = 2023-02-25
+++

Men da må også utlistingen ta høyde for dette. Men vi kan også lage vår egen template for RSS som filtrerer bort disse filene på en eller annen måte.

@timharek timharek mentioned this pull request Feb 25, 2023
@Kremtastic
Copy link
Contributor Author

Jeg tenkte at jeg skulle ikke fjerne noe fra filene, derfor endet jeg opp med to frontmatter på mange av de nyeste. Skal jeg dermed bare fjerne YAML (---) delen?

@timharek
Copy link
Member

timharek commented Mar 2, 2023

Jeg tenkte at jeg skulle ikke fjerne noe fra filene, derfor endet jeg opp med to frontmatter på mange av de nyeste. Skal jeg dermed bare fjerne YAML (---) delen?

@Kremtastic du kan godt fjerne YAML-frontmatter, hvis ikke vil formateringen se rar ut 😅
image

@Kremtastic Kremtastic dismissed timharek’s stale review March 30, 2023 16:06

changes completed

@Kremtastic Kremtastic merged commit 3ce6b1b into main Mar 30, 2023
@mathiash98 mathiash98 deleted the feature/admin-documents branch March 30, 2023 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants