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

Inicio de sesión - HTML y CSS #64

Merged
merged 15 commits into from
Apr 16, 2024
Merged

Inicio de sesión - HTML y CSS #64

merged 15 commits into from
Apr 16, 2024

Conversation

JLeonN
Copy link
Contributor

@JLeonN JLeonN commented Apr 7, 2024

No description provided.

@JLeonN JLeonN requested a review from francosang April 7, 2024 20:08
Agregar estado y funciones de manejo de errores al componente de inicio de sesión
console.log(window);
console.log(window.localStorage);

window.location.href = "/EstructuraDePerfil/1";
Copy link
Contributor

@franco-stuart franco-stuart Apr 10, 2024

Choose a reason for hiding this comment

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

La navegacion se hace con el hook, creo que se llamaba useNavigate() o algo asi.
Similar a como hiciste en otras paginas.

Copy link
Contributor

Choose a reason for hiding this comment

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

@JLeonN la navegación se hace así:

const navigate = useNavigate();

Antes que nada, acordate SIEMPRE de buscar ejemplos en el codigo.
Es decir, siempre que tengas que hacer algo, busca cosas similares en el codigo.
Pensá que otras cosas iguales o parecidas pueden existir.

Por ejemplo, ya habias hecho ejemplos de navegacion cuando vas del participante a la lista del titulo, o cuando hay un error.

navegar("/")

// ----Login:
// Usuario y contraseña 🔐
export async function InicioSesion(sesion) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Has programado 285647425 funciones, todas esmpiezan con minuscula y deben ser verbos o acciones.

Cambiar a inicarSesion.

const [errorInicioSesion, setInicioSesion] = useState(false);
const navegar = useNavigate();

const IniciarSesion = async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Igual que mi comentario antrerior, debe empezar en minuscula y que la primera palabra sea un verbo.

Comment on lines 41 to 43
// Tareas pendiente
// 1 navegar al inicio
// 2 guardar el token en la sesion ? buscar
Copy link
Collaborator

Choose a reason for hiding this comment

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

Borrar estos comentarios.

Comment on lines +16 to +17
// value={email}
// onChange={(e) => setEmail(e.target.value)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esto esta sin terminar.

Comment on lines +28 to +29
// value={contraseña}
// onChange={(e) => setContraseña(e.target.value)}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esto esta sin terminar.

<button
type="button"
className="btn btn-dark"
// onClick={IniciarSesion}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Esta sin terminar.

@@ -0,0 +1,56 @@
import { Link } from "react-router-dom";

const Registro = () => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Si te sirve de ayuda, acordate de estas preguntas:

// Que muestra mi pantalla?
//  - 
//
// Variables del estado:
//  - 
//
// Acciones de la pantalla:
//  - 
//
//  Pasos:
//  1. declarar los estados
//  2. asociar los estado al HTML
//  3. hacer una funcion para cada accion

Copy link
Collaborator

@francosang francosang left a comment

Choose a reason for hiding this comment

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

Daleeeee, maní quemado!

@JLeonN JLeonN requested a review from francosang April 16, 2024 21:54
@JLeonN JLeonN merged commit 3a57e47 into main Apr 16, 2024
1 check passed
@JLeonN JLeonN deleted the InicioDeSesión branch April 16, 2024 21:56
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.

3 participants