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

refactor: signup #65

Merged
merged 25 commits into from
Dec 13, 2021
Merged

refactor: signup #65

merged 25 commits into from
Dec 13, 2021

Conversation

mimoham24
Copy link
Contributor

Overview

What I've done

What I haven't done

How I tested

Which point I want you to review particularly

Memo

@codecov
Copy link

codecov bot commented Oct 17, 2021

Codecov Report

❗ No coverage uploaded for pull request base (feat/authentication-system@3913545). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@                      Coverage Diff                      @@
##             feat/authentication-system      #65   +/-   ##
=============================================================
  Coverage                              ?   16.61%           
=============================================================
  Files                                 ?      254           
  Lines                                 ?    43850           
  Branches                              ?        0           
=============================================================
  Hits                                  ?     7287           
  Misses                                ?    35528           
  Partials                              ?     1035           

@mimoham24 mimoham24 requested review from rot1024, HideBa and yk-eukarya and removed request for rot1024 October 25, 2021 00:29
internal/adapter/gql/resolver_mutation_user.go Outdated Show resolved Hide resolved
internal/adapter/http/user.go Outdated Show resolved Hide resolved
internal/usecase/interactor/user.go Outdated Show resolved Hide resolved
internal/usecase/interactor/user.go Show resolved Hide resolved
@mimoham24 mimoham24 requested a review from rot1024 October 31, 2021 21:38
pkg/user/builder.go Outdated Show resolved Hide resolved
pkg/user/auth.go Outdated Show resolved Hide resolved
pkg/user/user.go Outdated Show resolved Hide resolved
@mimoham24 mimoham24 requested a review from rot1024 November 4, 2021 20:24
@mimoham24 mimoham24 marked this pull request as ready for review November 5, 2021 21:38
pkg/user/user.go Show resolved Hide resolved
pkg/user/user.go Show resolved Hide resolved
pkg/user/user.go Outdated Show resolved Hide resolved
pkg/user/user_test.go Outdated Show resolved Hide resolved
pkg/user/user_test.go Outdated Show resolved Hide resolved
Copy link
Member

@rot1024 rot1024 left a comment

Choose a reason for hiding this comment

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

Also warning from golangci-lint should be fixed:

internal/adapter/http/user.go:49:2: SA9003: empty branch (staticcheck)
	if input.Email != nil {
	^

…th/reearth-backend into auth/signup

# Conflicts:
#	go.mod
#	go.sum
#	internal/usecase/interfaces/user.go
#	pkg/user/user_test.go
@mimoham24 mimoham24 requested a review from rot1024 November 18, 2021 05:44
pkg/user/builder.go Outdated Show resolved Hide resolved
pkg/user/initializer.go Show resolved Hide resolved
pkg/user/user.go Show resolved Hide resolved
pkg/user/user.go Show resolved Hide resolved
yk-eukarya and others added 4 commits November 29, 2021 13:20
* - 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>
* - send mail

* enhancement

* fix log
…th/reearth-backend into auth/signup

# Conflicts:
#	go.mod
#	internal/adapter/http/user.go
#	internal/app/public.go
#	internal/infrastructure/mongo/mongodoc/user.go
#	internal/infrastructure/mongo/user.go
#	internal/usecase/repo/user.go
#	pkg/user/builder.go
#	pkg/user/user.go
#	pkg/user/user_test.go
@mimoham24 mimoham24 requested a review from rot1024 December 8, 2021 07:59
pkg/user/builder.go Outdated Show resolved Hide resolved
pkg/user/builder.go Outdated Show resolved Hide resolved
@mimoham24 mimoham24 requested a review from rot1024 December 9, 2021 08:39
pkg/user/builder.go Outdated Show resolved Hide resolved
pkg/user/user.go Outdated Show resolved Hide resolved
@mimoham24 mimoham24 requested a review from rot1024 December 13, 2021 04:26
@mimoham24 mimoham24 merged commit 1e6d442 into feat/authentication-system Dec 13, 2021
@mimoham24 mimoham24 deleted the auth/signup branch December 13, 2021 07:55
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