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

Move association into Backend #3575

Merged
merged 12 commits into from
Nov 4, 2023
Merged

Move association into Backend #3575

merged 12 commits into from
Nov 4, 2023

Conversation

marcoacierno
Copy link
Member

@marcoacierno marcoacierno commented Oct 22, 2023

TODO

  • Prepare data for copying
  • Copy tests
  • Copy Stripe webhook
  • Copy Pretix webhook
  • Copy membership status check
  • Update isPythonItaliaMember GraphQL API in User to use the database to check membership
  • Fix broken add_*_paymentmethods
  • Implement queries/mutations to subscribe via stripe

Post deploy:

  • Update Stripe webhook URL
  • Update Pretix webhook URL
  • Figure out Monitoring
  • Copy data from prod association db to pycon

Pre-merge:

  • Disable all webhooks as a easy way to avoid data inconsistency when copying? not expecting anyone to subscribe over night anyway.

@vercel
Copy link

vercel bot commented Oct 22, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
pycon ❌ Failed (Inspect) Oct 31, 2023 8:44pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
association-frontend ⬜️ Ignored (Inspect) Visit Preview Oct 31, 2023 8:44pm

@marcoacierno marcoacierno changed the title Move association models into Backend Move association into Backend Oct 22, 2023
@codecov
Copy link

codecov bot commented Oct 22, 2023

Codecov Report

Merging #3575 (6b2038c) into main (21166a1) will decrease coverage by 2.70%.
Report is 825 commits behind head on main.
The diff coverage is 91.35%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3575      +/-   ##
==========================================
- Coverage   92.69%   90.00%   -2.70%     
==========================================
  Files         116      203      +87     
  Lines        2574     6262    +3688     
  Branches      247      681     +434     
==========================================
+ Hits         2386     5636    +3250     
- Misses        174      556     +382     
- Partials       14       70      +56     

@marcoacierno marcoacierno merged commit 0ddde5f into main Nov 4, 2023
11 of 18 checks passed
@marcoacierno marcoacierno deleted the association-be-into-pycon branch November 4, 2023 22:33
@marcoacierno marcoacierno mentioned this pull request Dec 9, 2023
20 tasks
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.

1 participant