iqrivas/savemymoney

SaveMyMoney is a web application created to help users to track monthly transactions such as expenses and incomes.

Web App

• SaveMyMoney is a web application created to help users to track monthly transactions such as expenses and incomes.
• This application was created using HTML, CSS, JavaScript (Vanilla) and Firebase.

Demo

Here you can find the app demo: https://savemymoney.vercel.app/

Built with

• HTML
• CSS
• JavaScript (Vanilla)
• Firebase

How does it work?

Made with ❤️ By:

Denisse Rivas | David Galeano

savemymoney's Issues

Feedback code review

Hola!

Hemos realizado la revisión de tu repositorio y hemos encontrado los siguientes puntos de mejora :)

Si tienes alguna inquietud o deseas solicitar una nueva revisión puedes comunicarte con Ana Belisa por Slack y enviar por el formulario nuevamente tu repositorio.

Pre-revisión

• El repositorio público

README.MD

• Descripción general del proyecto.
• Tecnologías implementadas en el proyecto, versión y objetivo con el cual se realizó la implementación.
• Enlace al demo.
• Enlace a documento con los requisitos del sistema.
• Nombre del coach que asignó y revisó el proyecto
• Información del autor del proyecto: nombre, usuario en Slack, enfoque (frontend, backend, ds).

GIT

• Los commits deben estar 100% en inglés.
• Los commits deben ser descriptivos y puntuales.
• No se permiten carpetas completas en un sólo commit con todo el proyecto.
• Debe existir al menos una rama adicional a main para desarrollo.

Revisión

Ruta del archivo: /main.js#L3-L11

var firebaseConfig = {
apiKey: "AIzaSyDcQAqiwbRyVgR3WdezSj1hpQoLAuBGG4M",
authDomain: "savemymoney-ec2ca.firebaseapp.com",
projectId: "savemymoney-ec2ca",
storageBucket: "savemymoney-ec2ca.appspot.com",
messagingSenderId: "442520639107",
appId: "1:442520639107:web:3bb6007192f315cd1c2db6",
measurementId: "G-RSLY3QK3FH"
};

Comentarios:
Recomendamos el uso de archivos de configuración, así se asegura la
correcta administración y la seguridad de las variables de entorno

---

Ruta del archivo: /main.js#L12-L16

// Initialize Firebase
firebase.initializeApp(firebaseConfig);

// Firestone DB Connection
const db = firebase.firestore();

/main.js#L31-L48
// Get all transactions
const getTransactions = () => db.collection('transactions').get();

// Save Transaction
const saveTransaction = (amount, category, date, name, type) => {
db.collection('transactions').doc().set({
amount,
category,
date,
name,
type

    });

}
// Delete transaction
const deleteTransaction = id => db.collection('transactions').doc(id).delete();

const onGetTransactions = (callback) => db.collection("transactions").orderBy("date", "desc").onSnapshot(callback);

Comentarios:
Tratemos de hacer uso de SOLID y usar el principio de una simple responsabilidad
ya que se nos hace facil leer un solo bloque con una funcion especifica,

---

Reviewer

Nombre: L. Raymundo Solís