Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

feat: send mail #86

Merged
merged 3 commits into from
Dec 6, 2021
Merged

feat: send mail #86

merged 3 commits into from
Dec 6, 2021

Conversation

yk-eukarya
Copy link
Contributor

No description provided.

@yk-eukarya yk-eukarya requested a review from rot1024 December 2, 2021 07:34
@yk-eukarya yk-eukarya self-assigned this Dec 2, 2021
@codecov
Copy link

codecov bot commented Dec 2, 2021

Codecov Report

Merging #86 (cd0f4b0) into auth/signup (de33ecd) will decrease coverage by 0.00%.
The diff coverage is 12.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           auth/signup      #86      +/-   ##
===============================================
- Coverage        16.52%   16.51%   -0.01%     
===============================================
  Files              250      250              
  Lines            43417    43441      +24     
===============================================
+ Hits              7173     7176       +3     
- Misses           35222    35241      +19     
- Partials          1022     1024       +2     
Impacted Files Coverage Δ
internal/usecase/interactor/user.go 0.99% <12.00%> (+0.99%) ⬆️

@yk-eukarya yk-eukarya changed the title - send mail feat: send mail Dec 2, 2021
@yk-eukarya yk-eukarya requested a review from mimoham24 December 2, 2021 07:47
internal/usecase/interactor/user.go Outdated Show resolved Hide resolved
internal/usecase/interactor/user.go Outdated Show resolved Hide resolved
internal/usecase/interactor/user.go Outdated Show resolved Hide resolved
@mimoham24 mimoham24 self-requested a review December 6, 2021 12:51
@yk-eukarya yk-eukarya merged commit 7201de1 into auth/signup Dec 6, 2021
@yk-eukarya yk-eukarya deleted the auth/password-reset-email branch December 6, 2021 13:46
mimoham24 added a commit that referenced this pull request Dec 13, 2021
* remove sugnup mutation

* new signup logic

* merge auth0 and signup logic

* lint

* revert signup mutation

* resolve notes

* remove password length

* resolve notes

* test cases

* change comparing method

* resolve notes

* match password

* update user data-layer

* feat: password reset (#79)

* - password reset

* - refactor

* - implement the memory repo

* - remove unnecessary structs

* - fix password reset token validation
- add unit test

* - fix linting

* - fix unit test

* - refactor

* - replace uuid lib
- rename validate method

* - fix go mod

* - fix go mod

* - enhance unit test

* - enhance unit test

* mock

Co-authored-by: rot1024 <aayhrot@gmail.com>

* feat: send mail (#86)

* - send mail

* enhancement

* fix log

* notes

* - add email design

* resolve

* fix user model

* refactor: move to bcrypt

* update verify password checks

Co-authored-by: yk-eukarya <81808708+yk-eukarya@users.noreply.github.com>
Co-authored-by: rot1024 <aayhrot@gmail.com>
Co-authored-by: yk <yk.eukarya@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants