Skip to content
This repository has been archived by the owner on Sep 29, 2024. It is now read-only.

feat: created verification email and token #269

Merged
merged 7 commits into from
Aug 12, 2024

Conversation

ArshiLamba
Copy link
Collaborator

@ArshiLamba ArshiLamba commented Aug 12, 2024

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Breaking Change
  • Documentation Update

Description

Related Tickets & Documents

  • Related Issue #
  • Closes #

QA Instructions, Screenshots, Recordings

Please replace this line with instructions on how to test your changes, a note
on the devices and browsers this has been tested on, as well as any relevant
images for UI changes.

Added/updated tests?

We encourage you to keep the code coverage percentage at 80% and above.

  • Yes
  • No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

[optional] Are there any post deployment tasks we need to perform?

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

Copy link

vercel bot commented Aug 12, 2024

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

Name Status Preview Comments Updated (UTC)
greenbites ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 9:36am
greenbites-server ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 9:36am

@github-actions github-actions bot added the Scope: Server This change affects the server. label Aug 12, 2024
Copy link
Owner

@caffeine-addictt caffeine-addictt left a comment

Choose a reason for hiding this comment

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

Also update the server/src/db/schema.ts to add 'verification' to the type casted strings of tokens table.

server/src/v1/user.ts Outdated Show resolved Hide resolved
server/src/utils/service/email/email.ts Show resolved Hide resolved
server/src/utils/service/email/email.ts Outdated Show resolved Hide resolved
server/src/v1/user.ts Outdated Show resolved Hide resolved
Repository owner deleted a comment from vercel bot Aug 12, 2024
Copy link

vercel bot commented Aug 12, 2024

Deployment failed with the following error:

Resource is limited - try again in 44 minutes (more than 100, code: "api-deployments-free-per-day").

Copy link

vercel bot commented Aug 12, 2024

Deployment failed with the following error:

Resource is limited - try again in 42 minutes (more than 100, code: "api-deployments-free-per-day").

@caffeine-addictt caffeine-addictt merged commit 3b1380c into main Aug 12, 2024
6 of 8 checks passed
@caffeine-addictt caffeine-addictt deleted the feature/user-settings-verification branch August 12, 2024 10:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Client This change affects the client. Scope: Server This change affects the server.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants