Pull Request - Memingo's#19
Conversation
…gistro y creacion de archivo databe.js
…se comenzo la segunda pantalla de registro con imagen y nombre de usuario
… de firestorage asociada con usuario y se pusieron iconos en la Navbar
…estore, storage y realtime
…r de cada post,post toma un tamaño fijo
This reverts commit c3553c5.
…ementos en el post
…rrores al iniciar sesión
| // Use IntelliSense to learn about possible attributes. | ||
| // Hover to view descriptions of existing attributes. | ||
| // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387 | ||
| "version": "0.2.0", | ||
| "configurations": [ | ||
| { | ||
| "type": "node", | ||
| "request": "launch", | ||
| "name": "Launch Program", | ||
| "skipFiles": [ | ||
| "<node_internals>/**" | ||
| ], | ||
| "program": "${workspaceFolder}\\src\\index.js" | ||
| } | ||
| ] | ||
| } No newline at end of file |
There was a problem hiding this comment.
Este archivo puede estar dentro de .gitignore
| "rules": { | ||
| ".read": "auth != null", | ||
| ".write": "auth != null" | ||
| } |
There was a problem hiding this comment.
Es muy cómodo tener las rules en este archivo pero también es mejor meterlo al .gitignore y no subirlo a github
| service cloud.firestore { | ||
| match /databases/{database}/documents { | ||
| match /{document=**} { | ||
| allow read, write: if request.auth != null; | ||
| } | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Lo mismo, mejor no subirlo
| "bulma": "^0.8.1", | ||
| "bulma-helper-spacing": "0.0.3", | ||
| "bulma-social": "^1.1.1", |
There was a problem hiding this comment.
Sabemos cuanto peso se agrego a nuestro proyecto con Bulma?, y si valió la pena en comparación con la cantidad de herramientas usadas (de bulma)
There was a problem hiding this comment.
No esta mal, pero es bueno saber cuanto peso agrega
| @@ -0,0 +1,79 @@ | |||
| # 😆 Memingo's, la red social de los memes. 💩 | |||
There was a problem hiding this comment.
Me encanta el nombre del app <3 😍
| .mainLogo{ | ||
| size: 12px; | ||
| margin:auto; | ||
| display: flex; | ||
| align-items: baseline; | ||
|
|
||
| } | ||
|
|
||
| .logoMemingos { |
There was a problem hiding this comment.
Los nombres de las clases se prefieren logo-memingos en vez de camelCase, dejemos el camel case para JS
| } | ||
|
|
||
| .userInfo { | ||
| align-items: center!important; |
There was a problem hiding this comment.
Prohibido usar !important hahaha.
| #textId { | ||
| width: 25%; | ||
| margin-top: 10px; | ||
| height: auto; | ||
| display: flex; | ||
| align-items: center; | ||
| justify-content: center; | ||
| font-weight: bold; | ||
| color: #cccccc; | ||
| } | ||
| #showNewImg { |
There was a problem hiding this comment.
Prefiramos clases sobre selectores de #
| import * as firebase from 'firebase'; | ||
| import 'firebase/auth'; | ||
| import database from '../src/database.js'; | ||
|
|
||
| describe('example', () => { | ||
| it('debería ser una función', () => { | ||
| expect(typeof example).toBe('function'); | ||
| let firebaseConfig = { | ||
| apiKey: "AIzaSyD6boxRV32Ld50HrVuWx2Fkl5ZlYQyAabk", | ||
| authDomain: "memingos-abeea.firebaseapp.com", | ||
| databaseURL: "https://memingos-abeea.firebaseio.com", | ||
| projectId: "memingos-abeea", | ||
| storageBucket: "memingos-abeea.appspot.com", | ||
| messagingSenderId: "299345545979", | ||
| appId: "1:299345545979:web:a1d0095b63dc2c48e5be5e", | ||
| measurementId: "G-7SHPQ7YX96" | ||
| }; | ||
| firebase.initializeApp(firebaseConfig); |
There was a problem hiding this comment.
Hermoso el hack para los tests
| describe('database.signUp', () => { | ||
| it('should sign up an account successfully', () => { | ||
| firebase.auth().onAuthStateChanged(function(user) { | ||
| expect(typeof user).toBe('object'); |
There was a problem hiding this comment.
Muy bien los test, pero en vez de toBe objeto, hay matchers que pueden comprobar que un objeto trae una llave especifica, como el uid, o podemos hacer:
expect(Object.keys(user).sort()).toEqual(['uid', 'email'].sort()); <- podemos comprobar todas las llaves o algunas.
|
Me encanta ver tanto código limpio y en orden, se pudo separar un poquito más entre DOM y Lógica, inténtenlo en el próximo proyecto html5, así como la semántica y los nombres de las clases css, pero felicidades Gran trabajo para tan poco tiempo Memingo's!! |
3 participants
Red social de los memes.