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

Create Example for StepZen using Vue and Supabase #54

Closed
wants to merge 4 commits into from
Closed

Create Example for StepZen using Vue and Supabase #54

wants to merge 4 commits into from

Conversation

Gokul-Ks
Copy link

@Gokul-Ks Gokul-Ks commented Oct 6, 2022

Submitting for hacktoberfest

Issue #45

Changes made: Added Example App for Vue App using Supabase and StepZen backend

Docs

Supabase : https://supabase.com/docs/guides/database/connecting-to-postgres
Vue: https://vuejs.org/guide/quick-start.html

@royderks
Copy link
Contributor

royderks commented Oct 6, 2022

Hello @Gokul-Ks thanks so much for adding an example integration! However a StepZen GraphQL API seems to be missing.

to create one, you can install the StepZen CLI:

npm i -g stepzen

And import the database from Supabase:

stepzen import postgresql

Please include the generated GraphQL schema in the PR and the data you inserted in Supabase.

Let me know if you have any questions :)

@Gokul-Ks
Copy link
Author

Gokul-Ks commented Oct 7, 2022

@royderks I have added the schema and data as requested, please merge my PR if it's all good

@royderks
Copy link
Contributor

@Gokul-Ks looks better already thank you.

Please remove your keys from the config.yaml, everyone looking at this PR is now able to use your authentication details.

Could you include a database dump of supabase, so people can use this to create their own database?

@@ -0,0 +1,44 @@
This is a [Next.js](https://nextjs.org/) project bootstrapped with [`create-next-app`](https://github.com/vercel/next.js/tree/canary/packages/create-next-app).

## Getting Started
Copy link
Contributor

Choose a reason for hiding this comment

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

To approve this PR the README should follow the format of the file TEMPLATE.md we have in this repository. All integrations have the same README format...

import Image from 'next/image'
import styles from '../styles/Home.module.css'

export default function Home() {
Copy link
Contributor

Choose a reason for hiding this comment

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

It doesn't seem like you are fetching data from the StepZen GraphQL API.. Could you implement some data fetching? Now it looks like just an API and an app that exist in the same directory, there's no communication between the two..

@@ -0,0 +1,5 @@
// Next.js API route support: https://nextjs.org/docs/api-routes/introduction

export default function handler(req, res) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think you need the default hello API route if you're not using it in the app

These are some examples of queries generated from the schema. Feel free to modify them or add your own.
"""
type Query {
getDemoList: [Demo]
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you include the .sql dump of the PostgreSQL database? That way other developers are able to clone this project, now they cannot..

@royderks
Copy link
Contributor

Hello @Gokul-Ks please have a look at the review comments. We're almost good to go, just some critical things are missing. Please let me know if you have any questions or need help..

@Gokul-Ks Gokul-Ks closed this by deleting the head repository Jan 11, 2024
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.

2 participants