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

feat: APIify database access #88

Merged
merged 13 commits into from
Jul 20, 2022
Merged

Conversation

thorwebdev
Copy link
Contributor

Following the cms-provider approach for database access in preparation of adding a Supabase DB provider.

@cla-bot
Copy link

cla-bot bot commented Jul 11, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@vercel
Copy link

vercel bot commented Jul 11, 2022

@thorwebdev is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Jul 11, 2022

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

Name Status Preview Updated
conference ✅ Ready (Inspect) Visit Preview Jul 20, 2022 at 3:54AM (UTC)

leerob
leerob previously approved these changes Jul 11, 2022
@leerob
Copy link
Member

leerob commented Jul 11, 2022

CleanShot 2022-07-11 at 06 59 19@2x

Also, can you sign the CLA? 🙏

@cla-bot
Copy link

cla-bot bot commented Jul 12, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@cla-bot
Copy link

cla-bot bot commented Jul 12, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@thorwebdev
Copy link
Contributor Author

Signed CLA sent and in the meantime I added in the supabase db provider.

@cla-bot
Copy link

cla-bot bot commented Jul 12, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@thorwebdev
Copy link
Contributor Author

thorwebdev commented Jul 12, 2022

I'm running into and issue with the image rendering: "ERROR Error: Failed to launch the browser process!
/tmp/chromium: error while loading shared libraries: libnss3.so: cannot open shared object file: No such file or directory
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md"

This answer on SO suggest to use node v 14, but not sure if that's the correct resolution. Any ideas?

image

@cla-bot
Copy link

cla-bot bot commented Jul 12, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@thorwebdev
Copy link
Contributor Author

thorwebdev commented Jul 12, 2022

Could it be related to the max serverless function size? But how is it working previously?

image

@cla-bot
Copy link

cla-bot bot commented Jul 14, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@cla-bot
Copy link

cla-bot bot commented Jul 14, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@thorwebdev
Copy link
Contributor Author

Okay, anyway, this issue seems unrelated to the changes in this PR, so I think we can go ahead and merge this in (it's running fine locally) and figure out the Vercel functions issue separately. I've opened a new issue for it here: #89

@cla-bot
Copy link

cla-bot bot commented Jul 19, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@cla-bot
Copy link

cla-bot bot commented Jul 19, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@cla-bot
Copy link

cla-bot bot commented Jul 20, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@leerob
Copy link
Member

leerob commented Jul 20, 2022

Might need to update the lock file

@cla-bot
Copy link

cla-bot bot commented Jul 20, 2022

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please follow the instructions at https://github.com/vercel/cla.

@thorwebdev thorwebdev requested a review from leerob July 20, 2022 03:49
@Deep-Codes
Copy link
Collaborator

Deep-Codes commented Sep 2, 2022

Register is failing 🤔 @thorwebdev

Check out the last deployment link of this PR

Screenshot 2022-09-02 at 6 34 12 PM

@thorwebdev
Copy link
Contributor Author

@Deep-Codes do you have any details on the error message of the API route?

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.

None yet

3 participants