Skip to content
This repository was archived by the owner on Jul 27, 2023. It is now read-only.

Conversation

@jonattan-infante
Copy link

Description

  • This PR includes the module connection to mariadb implementation Knex and the configuration in docker-compose

@omarefg
Copy link
Contributor

omarefg commented Aug 14, 2021

Can we please link the issue with the PR? 😁

@jonattan-infante
Copy link
Author

jonattan-infante commented Aug 14, 2021

@omarefg
Copy link
Contributor

omarefg commented Aug 14, 2021

Perfect, but what I do not see es the issue related to the PR. It's created?

Copy link
Contributor

@omarefg omarefg left a comment

Choose a reason for hiding this comment

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

Comments done 😁

@omarefg
Copy link
Contributor

omarefg commented Aug 14, 2021

@omarefg
Copy link
Contributor

omarefg commented Aug 14, 2021

There are conflicts with the package.json.

Copy link
Author

@jonattan-infante jonattan-infante left a comment

Choose a reason for hiding this comment

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

Thank you for your comments, I review all of them and I review all of them and I applied to changes suggestions. Please @omarefg @gh0stl1m, you can review again.

omarefg
omarefg previously approved these changes Sep 29, 2021
gh0stl1m
gh0stl1m previously approved these changes Oct 9, 2021
Co-authored-by: Santiago Sánchez Taborda <santiago.sanchez.t@gmail.com>
@jonattan-infante jonattan-infante dismissed stale reviews from gh0stl1m and omarefg via 43aaf0d November 9, 2021 04:36
@edanfesi edanfesi added this to the Sprint 4 (Users) milestone Nov 10, 2021
@@ -0,0 +1,6 @@
# `mariadb-connection-module`

> TODO: description

Choose a reason for hiding this comment

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

We have to add some explanation about this module, like the environments that the module use and so on.

@avanegasp avanegasp added the carry-over-4 Task that started in sprint 4 but it's not finished. label Nov 17, 2021
@edanfesi edanfesi added feature New feature or request and removed enhancement labels Nov 19, 2021
@jito-jito jito-jito added the weight:5 Something that need work label Nov 20, 2021
@edanfesi edanfesi added the backend Task for the backend side label Nov 22, 2021
@edanfesi edanfesi added the carry-over-5 task that started in sprint 5 but it's not finished. label Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backend Task for the backend side carry-over-4 Task that started in sprint 4 but it's not finished. carry-over-5 task that started in sprint 5 but it's not finished. feature New feature or request users-task Task related to users team weight:5 Something that need work

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

8 participants