Issue 71 - Add phone number field and migrations

codersforcauses / coexist

https://coexist-events.vercel.app

MIT License

6 stars 0 forks source link

Issue 71 - Add phone number field and migrations #73

Closed Connor2803 closed 1 month ago

Connor2803 commented 1 month ago

Change Summary

Added the phone number field into ExtendedUser and associated migration for it. Will go and add it to my /users/me endpoint once this is merged too.

Change Form

[x] The pull request title has an issue number
[x] The change works by "Smoke testing" or quick testing
[ ] The change has tests
[ ] The change has documentation

vercel[bot] commented 1 month ago

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

Name	Status	Preview	Comments	Updated (UTC)
coexist	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jul 15, 2024 2:57pm

Connor2803 commented 1 month ago

Almost ready to merge but am getting an error when creating a user with a phone number. It works if I create one without a number and add it after, but getting the following error on creation otherwise. Have asked @Divrr to take a look but I'll be trying to get it going too.

Connor2803 commented 1 month ago

Haven't changed much since the approval, but have tested again and all seems to be working, will merge now.