cl1ckname / micromango Goto Github PK

add optional field "liist" to manga preview response

i want to add wide cover for profile page

There is a danger moment about user registration

func (s *service) Register(ctx context.Context, req *pb.RegisterRequest) (*pb.UserResponse, error) {
	u, err := findByEmail(s.db, req.Email)
	if err != nil {
		if !errors.Is(err, gorm.ErrRecordNotFound) {
			return nil, err
		}
	}
	u.Email = req.Email
	u.PasswordHash = hashString(req.Password, s.salt)
	savedU, err := saveUser(s.db, u)
	if err != nil {
		return nil, err
	}
	if _, err := s.profile.Create(ctx, &profile.CreateRequest{
		UserId:   savedU.UserId.String(),
		Username: req.Username,
	}); err != nil {
		return nil, err
	}
	return &pb.UserResponse{
		UserId:   savedU.UserId.String(),
		Username: req.Username,
	}, nil
}

If registered successes but profile creating failed user won't be able to register again and also won't be able to visit his profile. I suppose there should be an algorithm for distributed transactions (actually idk). Advice me smthng?

one manga should have multiple genres, filter manga list according genres

Now I use network-mode: host for every service, this is not correct in my mind. Could someone please update docker-compose file?

Привет ~ヾ(・ω・)

У проекта совсем нет инструкций и руководств по развертыванию хотя бы dev-окружения, что усложняет возможно как-то пощупать исходный код без сильного погружения в него. Было бы замечательно добавить хотя бы отдельную WIP ветку с таким руководством. Добавить .env.example с набором переменных и значений для простого тестирования, рекомендации по миграциям и т.п.