search

kantinen / infoscreen

Slides til infoskærmen :art:
Do What The F*ck You Want To Public License
8 stars 29 forks source link

Fixed the quiz by updating to sheets v4 #77

Closed Lukires closed 3 years ago

Lukires commented 3 years ago

Skærmens quiz slide brugte før i tiden Google Sheet's API v3 til at hente spørgsmålene fra et sheet, men da apien er blevet discontinued (man skal nu bruge v4), og det alligevel virkede redundant ikke bare at have spørgsmålene som en json fil liggende lokalt, har jeg gjort netop det. Hvis der pludselig bliver interesse i at ændre eller tilføje spørgsmål hele tiden, så kunne det være det værd at gå til v4 I guess

BeneCollyridam commented 3 years ago

Grunden til at bruge Sheets var at alle kunne tilføje spørgsmål uden at skulle ind på Git

On Fri, Oct 29, 2021, 17:13 Lukas Schilling @.***> wrote:

Skærmens quiz slide brugte før i tiden Google Sheet's API v3 til at hente spørgsmålene fra et sheet, men da apien er blevet discontinued (man skal nu bruge v4), og det alligevel virkede redundant ikke bare at have spørgsmålene som en json fil liggende lokalt, har jeg gjort netop det. Hvis der pludselig bliver interesse i at ændre eller tilføje spørgsmål hele tiden, så kunne det være det værd at gå til v3.

You can view, comment on, or merge this pull request online at:

https://github.com/kantinen/infoscreen/pull/77 Commit Summary

File Changes

(2 files https://github.com/kantinen/infoscreen/pull/77/files)

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/kantinen/infoscreen/pull/77, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADYSEI2O2KJSLJWVWKZCPYLUJLB3FANCNFSM5G7R5TOQ .

Lukires commented 3 years ago

Grunden til at bruge Sheets var at alle kunne tilføje spørgsmål uden at skulle ind på Git … On Fri, Oct 29, 2021, 17:13 Lukas Schilling @.**> wrote: Skærmens quiz slide brugte før i tiden Google Sheet's API v3 til at hente spørgsmålene fra et sheet, men da apien er blevet discontinued (man skal nu bruge v4), og det alligevel virkede redundant ikke bare at have spørgsmålene som en json fil liggende lokalt, har jeg gjort netop det. Hvis der pludselig bliver interesse i at ændre eller tilføje spørgsmål hele tiden, så kunne det være det værd at gå til v3. ------------------------------ You can view, comment on, or merge this pull request online at: #77 Commit Summary - Fixed the quiz by using local JSON file instead of google sheets <68e692b> File Changes (2 files https://github.com/kantinen/infoscreen/pull/77/files) - M content/quiz.html https://github.com/kantinen/infoscreen/pull/77/files#diff-73672fc5973c8663621a2a0c6137d89c7473367111573798c56689745650a4cb (31) - A* content/res/quiz/questions.js https://github.com/kantinen/infoscreen/pull/77/files#diff-92462d93e51c9261c394839f2d4259e9540cd22064a7a7cf235c4b7f686bfdc5 (63) Patch Links: - https://github.com/kantinen/infoscreen/pull/77.patch - https://github.com/kantinen/infoscreen/pull/77.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#77>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADYSEI2O2KJSLJWVWKZCPYLUJLB3FANCNFSM5G7R5TOQ .

Siden v4 kræver en api key, har du nogen idéer til hvordan man smartest gør det, uden at expose api key'en?

Jeg tænkte man kunne sætte op en rest API på en anden server, som kunne fungerer som middle man, men det virker som meget arbejde.

Nogen idéer?

Lukires commented 3 years ago

Har implementeret sheets v4, spørgsmålene kan findes og opdateres her: https://docs.google.com/spreadsheets/d/1iH8BwyGJXnSjHuWmq6Bkx0PqT6TWhodrzWBQFrD6a4Y/edit?usp=sharing

Lukires commented 3 years ago

Vil dog gerne gøre opmærksom på at den tidligere (og denne) implementation har XSS injection. Tænker ikke det gør noget. Jeg tilføjede endda et spørgsmål der laver sjov med vi ikke har sanitized inputs:-)

BeneCollyridam commented 3 years ago

Fedt! Det gør ikke rigtigt noget, men vi burde nok fjerne det ¯_(ツ)_/¯