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

Database design #10

Merged
merged 18 commits into from
Sep 11, 2023
Merged

Database design #10

merged 18 commits into from
Sep 11, 2023

Conversation

brianmuks
Copy link
Collaborator

No description provided.

@vercel
Copy link

vercel bot commented Sep 5, 2023

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

Name Status Preview Comments Updated (UTC)
sparked-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 11, 2023 2:46pm

Copy link
Member

@OlivierJM OlivierJM left a comment

Choose a reason for hiding this comment

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

Let's keep using npm instead of having both with yarn or if you prefer yarn then we should remove package-lock.json.

Also I don't think animations are necessary I would instead recommend svgs like these https://undraw.co/illustrations

src/components/LandingPage/index.tsx Outdated Show resolved Hide resolved
src/components/LandingPage/index.tsx Outdated Show resolved Hide resolved
src/app/api/db/schema.tsx Outdated Show resolved Hide resolved
@brianmuks
Copy link
Collaborator Author

Let's keep using npm instead of having both with yarn or if you prefer yarn then we should remove package-lock.json.

Also I don't think animations are necessary I would instead recommend svgs like these https://undraw.co/illustrations

Noted. I will use svgs. I prefer yarn over npm 😄

@brianmuks
Copy link
Collaborator Author

@OlivierJM I have addressed the comments above !

@OlivierJM
Copy link
Member

@OlivierJM I have addressed the comments above !

Thanks.

@OlivierJM
Copy link
Member

@brianmuks this is error causing 2 actions to fail, to reproduce run yarn build on your local

Screenshot 2023-09-08 at 7 22 02 AM

@brianmuks
Copy link
Collaborator Author

brianmuks commented Sep 8, 2023 via email

@OlivierJM
Copy link
Member

@OlivierJM/sparked-next @.***> I noticed this error yesterday. It's now fixed. I resolved it on a new PR next-auth-config. I will merge the two branches .

can you merge it with this branch ? It will be easier.

@brianmuks
Copy link
Collaborator Author

brianmuks commented Sep 8, 2023 via email

@OlivierJM
Copy link
Member

Sure

@brianmuks can you merge that branch you mentioned that fixes this issue?

@brianmuks
Copy link
Collaborator Author

brianmuks commented Sep 11, 2023 via email

@brianmuks
Copy link
Collaborator Author

brianmuks commented Sep 11, 2023 via email

@OlivierJM
Copy link
Member

awesome, now all the checks are passing.

@OlivierJM
Copy link
Member

@brianmuks You can merge.

@brianmuks brianmuks merged commit d49f371 into SparkEdUAB:main Sep 11, 2023
6 checks passed
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