Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JCN 176 Crear Package Redis #1

Merged
merged 7 commits into from Oct 9, 2019
Merged

Conversation

gastonpereyra
Copy link
Contributor

@gastonpereyra gastonpereyra commented Oct 8, 2019

JCN-176-CREAR-PACKAGE-REDIS
JCN-176 Crear Package Redis

LINK AL TICKET
https://fizzmod.atlassian.net/browse/JCN-176

DESCRIPCIÓN DEL REQUERIMIENTO
1.1. Crear nuevo package utilizando el repositorio https://github.com/janis-commerce/redis
e instalandolo con @janiscommerce/package-generator

1.2. El package debe ofrecer wrapper de los siguientes métodos:

1.2.1. get, wrapper de hget

1.2.2. set, wrapper de hset

1.2.3. del, wrapper de hdel

1.3. Utilizar clase estática y cachear el cliente creado.

1.3.1. Crear el cliente en el primer método usado.

1.3.2. Utilizar las settings del valor redis utilizando el @janiscommerce/settings

1.3.3. Las settings tendrán el dato host para crear el cliente, ejemplo de settings:

{
	"redis": {
		"host": "the-redis-host"
	}
}

1.4. El package debe estar 100% testeado para todas sus funcionalidades y escenarios

1.5. El package debe estar 100% documentado para todas sus funcionalidades y escenarios, mostrando ejemplos de uso.

Comentarios extra:

Eliminar el uso de index.js, exportar directamente el archivo lib/redis.js.

DESCRIPCIÓN DE LA SOLUCIÓN
Se creó el paquete pedido, con tests y documentación.
Agregué que se puedan gettear todos los valores de una entidad de una vez.

@juanhapes juanhapes merged commit 0c5f672 into master Oct 9, 2019
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.

None yet

2 participants