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

Förslag på förbättringar #56

Open
pereriksson opened this issue Mar 29, 2021 · 2 comments
Open

Förslag på förbättringar #56

pereriksson opened this issue Mar 29, 2021 · 2 comments

Comments

@pereriksson
Copy link

pereriksson commented Mar 29, 2021

  • Vanilla JS Kmom01-02 blir alldeles för mycket upprepning. I js-kursen fick vi skapa och manipulera div:ar alldeles för många gånger. Låt oss få lära oss något nytt i kmom01-02 i denna kurs. Förslaget är att få börja jobba med Mithril tidigare i kursen. Det behöver inte bli svårt för studenten.
  • REST API:et borde förbereda studenten på hur API:er kommer se ut i arbetslivet. Token-baserad autentisering är (väl) vanligast där någon form av HTTP header sätts. Att sätta api_key i URL:en i vissa fall, i en form-parameter i vissa och i JSON request payload i andra fall blir bara rörigt.
@datalowe
Copy link

Jag har inte så mycket att tillägga här annat än att stämma in i att det vore bra om samma metod för att skicka med API-nyckel används oavsett typ av request. I issue:n som jag skrev så framgår ett tydligt konkret exempel på den rörigheten som @pereriksson beskriver kan uppstå i nuläget.

@pereriksson
Copy link
Author

Vill inte trampa på några tår, men så här uppdaterar man en artikel i ekonomisystemet Fortnox:

PUT /3/articles/FRPPLUS HTTP/1.1
Access-Token: 61cf63ae-4ab9-4a95-9db5-753781c4f41f
Client-Secret: 3Er4kHXZTJ
Content-Type: application/json
Accept: application/json

{
     "Article": {
         "ArticleNumber": "FRPPLUS",
         "Unit": "st"
     }
}

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

No branches or pull requests

2 participants