Sutty/hyperpop.js
5
0
Fork 0
Code Issues 2 Pull requests 2 Projects Releases Activity

headers personalizados #3 #4

Open
fauno wants to merge 3 commits from issue-3 into antifascista
pull from: issue-3
merge into: Sutty:antifascista
Conversation 2 Commits 3 Files changed 12 +82 -80
fauno commented 2022-11-01 17:26:23 +00:00
Owner
Copy link

@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional

@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional
fauno added 2 commits 2022-11-01 17:26:26 +00:00
fauno added 1 commit 2022-11-01 17:29:06 +00:00
Nulo reviewed 2022-11-01 19:44:02 +00:00
demo/demo.ts
@ -92,2 +103,2 @@
setupForm("#save-token", async () => {
localStorage.token = token();
setupForm("#token", () => {
console.log(headers());
Nulo commented 2022-11-01 19:44:02 +00:00
Owner
Copy link

Borrar console.lo

Borrar console.lo
Nulo requested changes 2022-11-01 19:47:28 +00:00
order.ts
@ -121,4 +121,1 @@
method: "POST",
headers: {
accesstoken: token,
"content-type": "application/json",
Nulo commented 2022-11-01 19:47:09 +00:00
Owner
Copy link

Se deberia aplicar el content-type siempre si lo requiere el endpoint

Se deberia aplicar el content-type siempre si lo requiere el endpoint
fauno commented 2022-11-01 20:48:58 +00:00
Author
Owner
Copy link

en el sitio le puse que siempre esté, no sé por qué en este endpoint específico se agrega y no en otros

en el sitio le puse que siempre esté, no sé por qué en este endpoint específico se agrega y no en otros
Nulo commented 2022-11-26 15:40:33 +00:00
Owner
Copy link

A mi entender es porque si se le pasa cosas por el body, en el resto no

A mi entender es porque si se le pasa cosas por el body, en el resto no
Nulo commented 2022-11-01 19:49:17 +00:00
Owner
Copy link

@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional

Yo usaría un type más específico como Record<"accesstoken" | string, string> (no se si esto específicamente funcionaría), pero tendrias que reimplementar headers() para que use un objeto plano

>@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional Yo usaría un type más específico como `Record<"accesstoken" | string, string>` (no se si esto específicamente funcionaría), pero tendrias que reimplementar headers() para que use un objeto plano
Nulo reviewed 2022-11-01 19:49:55 +00:00
demo/index.html
@ -34,0 +38,4 @@
<input id="user" name="user" />
<label for="pass">Pass: </label>
<input id="pass" name="pass" value="" />
Nulo commented 2022-11-01 19:49:55 +00:00
Owner
Copy link

Aclarar que es especifico para nuestro sistema

Aclarar que es especifico para nuestro sistema
fauno commented 2022-11-01 20:50:26 +00:00
Author
Owner
Copy link

bueno pero igual es una demostración de que podés customizar los headers

bueno pero igual es una demostración de que podés customizar los headers
Nulo commented 2022-11-06 22:44:59 +00:00
Owner
Copy link

No digo que lo saques, solo aclarar en la demo para que no sea confuso.

No digo que lo saques, solo aclarar en la demo para que no sea confuso.
fauno commented 2022-11-01 20:49:56 +00:00
Author
Owner
Copy link

@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional

Yo usaría un type más específico como Record<"accesstoken" | string, string> (no se si esto específicamente funcionaría), pero tendrias que reimplementar headers() para que use un objeto plano

el problema es que te limita a tener que mandar ese tipo específico, por eso se me complicó asociarle content-type, porque pueden venir tres tipos distintos

> >@Nulo no sé cómo redefinir el type HeadersInit para que tenga accesstoken opcional > > Yo usaría un type más específico como `Record<"accesstoken" | string, string>` (no se si esto específicamente funcionaría), pero tendrias que reimplementar headers() para que use un objeto plano el problema es que te limita a tener que mandar ese tipo específico, por eso se me complicó asociarle content-type, porque pueden venir tres tipos distintos
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin issue-3:issue-3
git checkout issue-3

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout antifascista
git merge --no-ff issue-3
git checkout issue-3
git rebase antifascista
git checkout antifascista
git merge --ff-only issue-3
git checkout issue-3
git rebase antifascista
git checkout antifascista
git merge --no-ff issue-3
git checkout antifascista
git merge --squash issue-3
git checkout antifascista
git merge --ff-only issue-3
git checkout antifascista
git merge issue-3
git push origin antifascista
Sign in to join this conversation.
Reviewers
No reviewers
Nulo
Labels
Clear labels
  
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Sutty/hyperpop.js#4
No description provided.
Delete branch "issue-3"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?