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

feat: update tag #49

Merged
merged 19 commits into from
Oct 23, 2021
Merged

feat: update tag #49

merged 19 commits into from
Oct 23, 2021

Conversation

mimoham24
Copy link
Contributor

@mimoham24 mimoham24 commented Sep 15, 2021

Overview

update tag

What I've done

I've created a mutation to update a tag

What I haven't done

How I tested

dummy data

Which point I want you to review particularly

Memo

… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/resolver_mutation_tag.go
#	internal/infrastructure/memory/tag.go
#	internal/usecase/interactor/tag.go
#	internal/usecase/interfaces/tag.go
#	schema.graphql
@mimoham24 mimoham24 changed the title Tag/update group label feat: rename tag group Sep 15, 2021
@codecov
Copy link

codecov bot commented Sep 15, 2021

Codecov Report

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

Impacted file tree graph

@@              Coverage Diff              @@
##             tag-system      #49   +/-   ##
=============================================
  Coverage              ?   16.02%           
=============================================
  Files                 ?      270           
  Lines                 ?    44968           
  Branches              ?        0           
=============================================
  Hits                  ?     7208           
  Misses                ?    36747           
  Partials              ?     1013           

schema.graphql Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
@mimoham24 mimoham24 changed the title feat: rename tag group feat: update tag Sep 22, 2021
… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	schema.graphql
… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/resolver_mutation_tag.go
#	internal/usecase/interactor/tag.go
#	internal/usecase/interfaces/tag.go
#	schema.graphql
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.

Currently we can only update tag groups, but TagItem should also be able to be updated.

schema.graphql Outdated Show resolved Hide resolved
internal/usecase/interactor/tag.go Outdated Show resolved Hide resolved
internal/usecase/interactor/tag.go Outdated Show resolved Hide resolved
… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/usecase/interactor/tag.go
#	internal/usecase/interfaces/tag.go
#	schema.graphql
… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/gqlmodel/convert_tag.go
… into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/gqlmodel/convert_tag.go
@mimoham24 mimoham24 merged commit 820acea into tag-system Oct 23, 2021
@mimoham24 mimoham24 deleted the tag/update-group-label branch October 23, 2021 22:56
mimoham24 added a commit that referenced this pull request Oct 25, 2021
* feat: tag system domain models (#39)

* feat: tag system domain models

* refactor: * add tag interface * tag -> group and tag->item conversation

* testing: generate test cases for the tagID

* resolve notes

* fix unit tests errors

* add NewId test code
fix NewId func

* add more test cases
refactor some parts

* feat: tag system data-layer (mongo) (#44)

* feat: tag system data-layer (mongo)

* remove len > 0 check

* goimport

* Update pkg/tag/group_builder.go

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

* Update pkg/tag/item_builder.go

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

* rename itemFrom and groupFrom funcs

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

* feat: create tag group and tag item (#45)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* resolve notes

* generate new models

* feat: memory infrastructure (#46)

* refactor: implement memory infrastructure

* test: implement memory infrastructure test cases

* test: fix FindByScene test case

* feat: attach/detach tag from layer (#50)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: introduce tags to layers

* feat: attach/detach tags from layers

* fix imports

* refactor: resolve notes

* test: test units for tags

* refactor: resolve notes

* feat: attach/detach tag item from group (#52)

* refactor: transform group tags list to reference

* feat: attach/detach tags

* refactor: use params as use-case input

* test: mongodoc testing (#61)

* test: mongodoc testing

* resolve notes

* feat: remove tag (#58)

* feat: remove a tag (init)

* feat: remove tag

* feat: remove tag usecase and infra

* resolve notes

* feat: tag system queries (#54)

* feat: tag system queries

* resolve notes

* feat: update tag (#49)

* tag item and group schema

* feat: creat tags (GQL schema)

* tag items and tag groups resolvers

* datalayer (dummy memory) and usecases

* receive list by reference

* check if nil for list

* feat: rename tag group

* refactor: rename -> update

* resolve notes

* resolve notes

* change find by id func return type

* Merge branch 'tag-system' of https://github.com/reearth/reearth-backend into tag/update-group-label

# Conflicts:
#	internal/adapter/gql/generated.go
#	internal/adapter/gql/gqlmodel/convert_tag.go

* fix testing

* resolve notes

* resolve notes

* resolve notes

Co-authored-by: HideBa <baba.papa1120.ba@gmail.com>
Co-authored-by: rot1024 <aayhrot@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
3 participants