Greenstand / Greenstand/treetracker-query-api

Extend the BaseRepository to support: delete

Open
#24 3 comments 0 reactions 1 assignee Assigned to @MundhadHarsh View on GitHub
good first issue Node.js
Dominant language
TypeScript
Stars
18
Forks
65
PR merge metrics
No merged PRs in 30d

Description

The function is for deleting the entity in the database.

The function signature should like:

```
delete(object: T>: T
```

Some the subclass can do:

```
TokenRepository.delete(token)
```

Possible edge cases:

* Can not find the entity by id. (404 with error message)
* Maybe there are some cases that database will throw error like, foreign key violation, we should translate this type of exception to : (500, database exception)
* Our rule for deletion is that we don't really delete the record, but set the `active` field to `false`

---

Some hints:

- Please read our readme for more information/guide/tutorial.
- Here is [an engineering book](https://greenstand.gitbook.io/engineering/) in Greenstand.
- To know more about our organization, visit our [website](https://greenstand.org).
- If you want to join the slack community (some resources need the community member's permission), please leave your email address.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.