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

Change/upgrade #31

Merged
merged 26 commits into from
May 15, 2019
Merged

Change/upgrade #31

merged 26 commits into from
May 15, 2019

Conversation

blackshadev
Copy link
Collaborator

@blackshadev blackshadev commented May 6, 2019

Nieuwe upgrade naar 5.8 vanaf de locale branch

Closes #24

@robinklaassen
Copy link
Owner

robinklaassen commented May 6, 2019

To do:

  • Check registration forms
  • Check iDeal payments using test key
  • Check emails
  • ...

@blackshadev
Copy link
Collaborator Author

Test report

Registratie formulieren

Zowel /register-participant als /register-member geprobeert
Ik heb meerdere malen een participant succesvol aangemaakt, daarna op zijn account ingelogd , beetje rond geklikt. Works like a charm. Password reset werkt

Check mails

Ik krijg de volgende mails tijdens het testen wachtwoord gereset, bevestiging van inschrijving, nieuwe deelnemer. Seems legit

Mollie

Ik heb met een test API een betaling gedaan, het meeste werkt, maar de webhook komt niet aan, dit is uiteraard omdat hij lokaal draait. Ik durf de gok te wagen en ik zie geen manier om dit te testen aangezien molie niet de betalingsID vrijgeeft aan mij. Als je het HEEEUL zeker wil weten kunnen we het naar een requestbin duwen en de request dan lokaal replay-en @robinklaassen wat denk jij?

@robinklaassen
Copy link
Owner

@blackshadev goed bezig! Het Mollie stuk vertrouw ik wel inderdaad.

Ik heb nog wat verder getest, wat dingetjes gefixt met de model binding (implicit in de routes werkt prima, als de namen maar overeenkomen. Dus niet {users} in de url, maar {user} als je in je controller method vervolgens ook User $user gebruikt), en wat nieuwe seeds toegevoegd.

Vanwege die extra seeds zou ik zeggen: closes #1

@robinklaassen
Copy link
Owner

Ik zit heel hard te denken of we nog iets vergeten zijn te testen, maar er komt nog niks bovendrijven :) ik geef het nog een dagje ofzo en dan merge ik 'm. Daarna kijken naar deployment.

@blackshadev
Copy link
Collaborator Author

@robinklaassen Ik weet niet hoe ik je persé kan helpen met de deploy scripts aangezien jij alle credentials hebt (en dat is ook logisch). Als je er behoefte aan heb hulp kunnen we ook een avondje afspreken om het samen te doen.

@robinklaassen
Copy link
Owner

@blackshadev geen probleem, ik denk dat ik er zelf wel uitkom. Ik zou zeggen ga lekker door met je werk aan Mollie en het rollensysteem, en dan houden we elkaar wel op de hoogte via github.

Ik zal trouwens nu even mailen naar het bestuur dat we graag een sparringspartner willen voor het rollensysteem.

@robinklaassen
Copy link
Owner

robinklaassen commented May 13, 2019

Gebroken Excel exports gevonden:

  • MembersController@export
  • ParticipantsController@export

@blackshadev
Copy link
Collaborator Author

done

@robinklaassen
Copy link
Owner

Yes, netjes gedaan @blackshadev !

Copy link
Owner

@robinklaassen robinklaassen left a comment

Choose a reason for hiding this comment

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

Topwerk

@blackshadev
Copy link
Collaborator Author

splut 2 vliegen, 1 klap!

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.

Laravel upgrade
2 participants