[E-graphQL < T- map] Add graphQL support for map - Githubissues

memgraph / mage

MAGE - Memgraph Advanced Graph Extensions :crystal_ball:

Apache License 2.0

244 stars 24 forks source link

[E-graphQL < T- map] Add graphQL support for map #362

Closed mpintaric55334 closed 11 months ago

mpintaric55334 commented 11 months ago

Description

Please briefly explain the changes you made here.

Pull request type

[ ] Bugfix
[ ] Algorithm/Module
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Related issues

Delete if this PR doesn't resolve any issues. Link the issue if it does.

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

[ ] Core algorithm/module implementation
[ ] Query module implementation
[ ] Unit tests
[ ] End-to-end tests
[ ] Code documentation
[ ] README short description
[ ] Documentation on memgraph/docs
[ ] Update GQLALchemy signatures in query builder using query module signature generator

######################################

mpintaric55334 commented 11 months ago

@antoniofilipovic @imilinovic please when reveiwing double check SetKey, removeKey, and removeKeys. Here the warcrimes for std::string_view were solved, and for removing a recursive map instead of bool was added