Skip to content

Conversation

@EliuX
Copy link
Contributor

@EliuX EliuX commented Apr 7, 2020

In this PR we create a new Data Access Layer (DAL) for Cosmos DB. It also makes room for other projects, namely the planned time-tracker-events, to use this DAL as well. Therefore, all implementations of the DAL were moved out to a new namespace called commons. This way any resource or utility that is common to multiple applications would be stored in this namespace.
The new DAL allows:
To handle items with a repository:

  • Create items
  • Partial update items
  • Completely update items
  • Delete items
  • find one item
  • find multiple items: allows pagination

To handle containers:

  • Create a container, based on a definition.
  • Delete a container

Tests were created to cover 100% of all the functional code of these new features.

@EliuX EliuX requested review from Angeluz-07 and enriquezrene April 7, 2020 20:57
@EliuX EliuX force-pushed the feature/refactor-dal-for-cosmos-sql-api#50 branch 2 times, most recently from f720b7a to 7fba702 Compare April 7, 2020 22:02
@EliuX EliuX force-pushed the feature/refactor-dal-for-cosmos-sql-api#50 branch 2 times, most recently from e6138b8 to cc1071d Compare April 8, 2020 16:22
@EliuX EliuX force-pushed the feature/refactor-dal-for-cosmos-sql-api#50 branch from cc1071d to 5036fa3 Compare April 8, 2020 17:28
@EliuX EliuX force-pushed the feature/refactor-dal-for-cosmos-sql-api#50 branch from 5036fa3 to 6b17a36 Compare April 8, 2020 18:10
@EliuX EliuX merged commit 84b1e70 into master Apr 8, 2020
@EliuX EliuX deleted the feature/refactor-dal-for-cosmos-sql-api#50 branch April 8, 2020 18:22
@EliuX EliuX self-assigned this Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants