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] Lav async kald til opskriv venteliste #481

Open
Rotendahl opened this issue Mar 13, 2020 · 3 comments
Open

[Feature] Lav async kald til opskriv venteliste #481

Rotendahl opened this issue Mar 13, 2020 · 3 comments
Labels
C Priority C feature ny oplevelse eller adfærd

Comments

@Rotendahl
Copy link
Contributor

Overordnet beskrivelse af funktionen
Handlinger som f.eks opskriv venteliste, tilmeld event, osv. laver en fuld post hver gange så man mister sin plads på siden.

Jeg foreslår at vi sætter noget async kode op så det kan gøres uden en refresh.
Undersøg evt. om https://rxjs-dev.firebaseapp.com/guide/overview ville passe godt ind.

Ellers kan vi bare bruge promises osv. evt. gennem https://github.com/axios/axios så det virker på IE11.

Ting der skal gøres

  1. Vælg fremgangsmåde (axios/rxjs) lav et setup og evt. en wrapper.
  2. Opdater opskriv venteliste kode
  3. Sørg for errorhandling
  4. Test
  5. Send PR

Så kan vi efter setup er lavet se hvilke andre steder i koden vi skal have det samme.

@Rotendahl Rotendahl added the feature ny oplevelse eller adfærd label Mar 13, 2020
@Rotendahl
Copy link
Contributor Author

@nallic var det ikke dig der viste mig RxJs?
Jeg kender det ikke, har du noget at sige om det?

@JakobLibak JakobLibak added the C Priority C label Mar 22, 2020
@benj5378
Copy link
Contributor

Se evt. #394

@JakobLibak
Copy link
Member

#394 blev lukket for 1½ år siden. Hvad foreslår du at vi gør for at komme videre? @benj5378

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C Priority C feature ny oplevelse eller adfærd
Projects
None yet
Development

No branches or pull requests

3 participants