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

feat: remove tag #58

Merged
merged 6 commits into from
Oct 23, 2021
Merged

feat: remove tag #58

merged 6 commits into from
Oct 23, 2021

Conversation

mimoham24
Copy link
Contributor

Overview

remove tag rules:

  • can't remove a tag non-empty tag groups
  • a removed tag group need to be detached from the layer
  • a removed tag item should be detached from both tag group and layer

What I've done

creat:

  • use-cases
  • gql mutaions
  • datalayers

What I haven't done

How I tested

dummy data

Which point do I want you to review particularly

Memo

… into tag/remove-tag

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

# 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
@mimoham24 mimoham24 changed the title Tag/remove tag feat: remove tag Sep 28, 2021
@codecov
Copy link

codecov bot commented Sep 28, 2021

Codecov Report

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

Impacted file tree graph

@@              Coverage Diff              @@
##             tag-system      #58   +/-   ##
=============================================
  Coverage              ?   16.00%           
=============================================
  Files                 ?      269           
  Lines                 ?    44442           
  Branches              ?        0           
=============================================
  Hits                  ?     7112           
  Misses                ?    36321           
  Partials              ?     1009           

Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

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

LGTM 👍

internal/usecase/interactor/tag.go Outdated Show resolved Hide resolved
schema.graphql Outdated Show resolved Hide resolved
internal/usecase/interactor/tag.go Show resolved Hide resolved
@mimoham24 mimoham24 merged commit 27b11b0 into tag-system Oct 23, 2021
@mimoham24 mimoham24 deleted the tag/remove-tag branch October 23, 2021 21:54
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