Social Network equipo 14 by fabi-pharoxysme · Pull Request #33 · Laboratoria/CDMX009-Social-Network

fabi-pharoxysme · 2020-05-05T22:35:01Z

Pull request de proyecto Social-Network
Equpo 14 :
Shei Miceli
Astrid Gutierrez
Fabiola Ruiz

…n un poco de interaccion, permite pasar de una pagina a otra por medio de los botones de registrarse e iniciar sesion

…iciar sesion sin problema

…nera se agrego una pagina de inicio

…twork

Se agregan imagenes de las pantallas en la aplicación

Intro Bake Recipes Book

Se agregan imagenes de la versión final

Cambios en Readme

Cambios al readme

Cambios al Readme

add button delete function in profile

reloadercf

Hay mucho que mejorar pero siguiendo las recomendaciones y documentándose sera posible una mejora continua, creo que elegir un framework al inicio y probarlo puede que sea la mejor forma para decidir si se queda o se desinstala, mantener 2 frameworks definitivamente esta mal, algo que me hubiera gustado ver seria testing, es importante empezarlo paralelamente el testing, lo que más me gusto de su proyecto es la estructura y ver que en su mayoria implementaron buenas practicas.

reloadercf · 2020-05-10T02:08:39Z

+export const db = firebase.firestore();
+// Authentication
+const usersRef = db.collection('users');
+const auth = firebase.auth();


Me agrada que tengan la buena practica de nombran const a las variables que no cambian ;)

reloadercf · 2020-05-10T02:10:08Z

+}
+export const login = (email, password) => {


Suggested change

}

export const login = (email, password) => {

}

export const login = (email, password) => {

Sugerencia

reloadercf · 2020-05-10T02:12:04Z

+export const signup = (email, password) => {
+  auth.createUserWithEmailAndPassword(email, password)
+    .then((snap => {
+      console.log(snap);


En una versión de construcción esta bien tener los console.log( ) pero en una versión de producción (y más para revisión) se deben de quitar.

Suggested change

console.log(snap);

reloadercf · 2020-05-10T02:12:47Z

+      console.log(snap);
+      saveUser(snap.user);
+      let mail = snap.user.email;
+      alert(`${mail} registro exitoso!`);


No se recomienda el uso de alerts porque suele ser confuso para el usuario.

reloadercf · 2020-05-10T02:13:04Z

+    .catch((error => {
+      let errorCode = error.code;
+      let errorMessage = error.message;
+      alert(`lo sentimos: ${errorCode} ${errorMessage}`)


otro alert

Suggested change

alert(`lo sentimos: ${errorCode} ${errorMessage}`)

reloadercf · 2020-05-10T03:30:45Z

+  border: 0px;
+}
+
+.imgSignUp{


Suggested change

.imgSignUp{

.img-sign-up{

reloadercf · 2020-05-10T03:31:02Z

+  border-radius: 92px;
+}
+
+.logoSignUp{


Suggested change

.logoSignUp{

.logo-sign-up{

reloadercf · 2020-05-10T03:35:12Z

+    root.innerHTML = `<div id="navBar">
+    <img class="is-rounded"  src="https://uxrecipe.github.io/img/uxrecipe-logo.png" id="logo">
+    <div class="dropdown">
+    <button class="dropbtn"><i class="fas fa-bars" aria-hidden="true"></i></button>
+    <div class="dropdown-content">
+    <a href="#"><i class="fas fa-bell" aria-hidden="true"></i> Notificaciones</a>
+    <a href="#"><i class="fas fa-bookmark" aria-hidden="true"></i> Guardado</a>
+    <a href="#"><i class="fas fa-wrench" aria-hidden="true"></i> Configuracion</a>
+    </div>
+    </div>
+    </div>


identación

Suggested change

root.innerHTML = `<div id="navBar">

<img class="is-rounded" src="https://uxrecipe.github.io/img/uxrecipe-logo.png" id="logo">

<div class="dropdown">

<button class="dropbtn"><i class="fas fa-bars" aria-hidden="true"></i></button>

<div class="dropdown-content">

<a href="#"><i class="fas fa-bell" aria-hidden="true"></i> Notificaciones</a>

<a href="#"><i class="fas fa-bookmark" aria-hidden="true"></i> Guardado</a>

<a href="#"><i class="fas fa-wrench" aria-hidden="true"></i> Configuracion</a>

</div>

</div>

</div>

root.innerHTML =

`<div id="navBar">

<img class="is-rounded" src="https://uxrecipe.github.io/img/uxrecipe-logo.png" id="logo">

<div class="dropdown">

<button class="dropbtn">

<i class="fas fa-bars" aria-hidden="true"></i>

</button>

<div class="dropdown-content">

<a href="#"><i class="fas fa-bell" aria-hidden="true"></i> Notificaciones</a>

<a href="#"><i class="fas fa-bookmark" aria-hidden="true"></i> Guardado</a>

<a href="#"><i class="fas fa-wrench" aria-hidden="true"></i> Configuracion</a>

</div>

</div>

</div>

reloadercf · 2020-05-10T03:37:14Z

+    </div>
+    </div>
+    </div>
+    <br>


Cada vez que se usa br muere un gatito :´´( no usen br mejor usen salto de párrafo

Suggested change

<br>

<p/>

reloadercf · 2020-05-10T03:38:48Z

+    const profile = document.querySelector("#profile");
+    profile.onclick = postPage;
+    const newPost = document.querySelector('#newPost');


Si la variable cambia el valor es mejor usar let

Suggested change

const profile = document.querySelector("#profile");

profile.onclick = postPage;

const newPost = document.querySelector('#newPost');

let profile = document.querySelector("#profile");

profile.onclick = postPage;

let newPost = document.querySelector('#newPost');

Sheiki1 and others added 25 commits April 15, 2020 09:23

se agrego toda la maquetacion del registro y de iniciar sesion tambie…

de229ff

…n un poco de interaccion, permite pasar de una pagina a otra por medio de los botones de registrarse e iniciar sesion

se agregaron estilos con bulma y ya se puede cambiar de registro a in…

627ac60

…iciar sesion sin problema

se agrego la opcion de conectarse por medio de facebook y de igual ma…

7ec833b

…nera se agrego una pagina de inicio

add post page, MVC, login and sign up with email and password

12e91bd

bajando cambios

25a9612

add login with fb and google

faa827e

g

6f74b07

Merge branch 'master' of https://github.com/Sheiki1/CDMX009-Social-Ne…

75cc426

…twork

Se agregan imagenes

ff8c923

Se agregan imagenes de las pantallas en la aplicación

add post function

6bbcc69

change everything

6cd37ec

change everything

5e2d17a

Add new post, delete post and profile functions

0c6ffd5

Bake Recipes Book

77ac6ff

Intro Bake Recipes Book

Se agregan imagenes

13f301f

Se agregan imagenes de la versión final

Cambios en Readme

730d57e

Cambios en Readme

ad objetivos de aprendizaje

53711bb

agregue mis objetivos de aprendizaje

ff63dd9

last change

8d20375

ultimo cambio se objetivos de aprendizaje

26aa2cc

Cambios al readme

778689f

Cambios al readme

Cambios al Readme

dcb5130

Cambios al Readme

Cambios al Readme

6fa57d4

Cambios al Readme

add button delete function in profile

a41f824

Merge pull request #1 from Sheiki1/fabi-branch

0073f65

add button delete function in profile

reloadercf suggested changes May 10, 2020

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Social Network equipo 14#33

Social Network equipo 14#33
fabi-pharoxysme wants to merge 25 commits into
Laboratoria:masterfrom
Sheiki1:master

fabi-pharoxysme commented May 5, 2020

Uh oh!

reloadercf left a comment

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

reloadercf May 10, 2020

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

fabi-pharoxysme commented May 5, 2020

Uh oh!

reloadercf left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants