Skip to content
This repository has been archived by the owner on Jan 3, 2025. It is now read-only.

Fix more bugs found during testing on staging #286

Merged
merged 29 commits into from
Oct 26, 2023
Merged

Fix more bugs found during testing on staging #286

merged 29 commits into from
Oct 26, 2023

Conversation

FinnIckler
Copy link
Member

Had to merge the second PR to continue testing, so here are more fixes that came up when testing

@FinnIckler
correctly import the polling
5a6d3f8
@FinnIckler
unhardcode the routes
9438e8c
@FinnIckler
at v2 to competitions
2de612c
@FinnIckler
add cors headers to lambda
8ff07dc
@FinnIckler
correctly add attendee_id to polling url
992b8c8
@FinnIckler
remove unnecessary eslint comment
829d5b1
@FinnIckler
fix event_change_deadline being null breaking the validate events check
d9bc3cf
@FinnIckler
corrected the headers in lambda
c440777
@FinnIckler
Correctly assemble attendee_id for polling
776fc62
@FinnIckler
correctly return competing_status wrapped in { competing }
fb3202b
@FinnIckler
correctly read competition info from the right WCA_HOST
2905e5f
@FinnIckler
add null guard to data
219cc8a
@FinnIckler
remove payment_init path as it is local
1162e22
@FinnIckler
correctly find competition in payment_init
88c4619
@FinnIckler
check if registration is nil when trying to generate a payment ticket
eb11939
@FinnIckler
use EnvConfig when accessing WCA_HOST
b8f593f
@FinnIckler
add https to url
40c8e87
@FinnIckler
add missing JSON.stringify
232ee2b
@FinnIckler
correctly name paymentId to id
e4d33c7
@FinnIckler
update correct name for payment_id
42f4f2b
@FinnIckler
send the data in a ruby_snake_case
1da496d
@FinnIckler
correctly set user
65220e8
@FinnIckler
put attendee_id as get parameter
8e6e368
@FinnIckler
create update_payment_status route
c9c8d3d
@FinnIckler
return payment_status on the frontend
4e6ee29
@FinnIckler
run rubocop
8824adf
@FinnIckler
add WCA_HOST to dev and test
45c1082
@FinnIckler
Copy link
Member Author

@thewca-bot deploy staging

@FinnIckler
Copy link
Member Author

@thewca-bot deploy staging

@FinnIckler FinnIckler marked this pull request as ready for review October 26, 2023 00:16
@FinnIckler FinnIckler merged commit 61f9715 into main Oct 26, 2023
1 check passed
@FinnIckler FinnIckler deleted the fix/more-staging branch October 26, 2023 00:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
No open projects
WST Project Management
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@FinnIckler