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

Mvpsdvlpr password generator #369

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

mvpsdvlpr
Copy link

⭐ Descripción

Agrego mi Password Generator en el directorio src/pages/entry/mvpsdvlpr con componentes en src/components/mvpsdvlpr

🚀 Tipo de cambio

  • Arreglo de bug (cambios que no rompen nada y resuelven un problema).
  • Nueva característica (cambios que no rompen nada y agregan funcionalidad).
  • Cambio radical (arreglo o característica que podría causar que una funcionalidad no funcione como se espera).
  • Este cambio requiere actualización de documentación.

🤔 ¿Cómo ha sido probado este cambio?

Esta aplicación no necesita configuración previa, solo hace falta acceder a ella
y las pruebas que se realizaron fueron

  • Gerenacion de password al momento de mover slider.
  • Aumento de caracteres o disminucion de estos al manipular slider.
  • Modificacion contrasena al momento de seleccionar checkbox de uppercase, lowercase, agregando tipo de caracter a nueva contrasena.
  • Modificacion contrasena al momento de seleccionar checkbox de symbols agregando tipo de caracter a nueva contrasena.
  • Modificacion contrasena al momento de seleccionar checkbox de Numbers agregando tipo de caracter a nueva contrasena.
  • Al presionar el icono de copiar en un item del historial, se debe copiar la contraseña de ese item.

📝 Lista de verificación:

  • Mi código sigue las reglas de este proyecto *.
  • He creado mis propias clases / ids y he comprobado que no afectan a otros diseños de la comunidad *.
  • He realizado una auto-revisión de mi propio código *.
  • He comentado mi código, particularmente en las áreas difíciles de entender.
  • Mis cambios no generan nuevos avisos *.
  • No he instalado nuevos módulos con npm install o yarn install *.
  • La solución es de mi autoría y no he copiado código de otros participantes *.
  • No modifiqué ningún otro archivo más que los míos *.

@vercel
Copy link

vercel bot commented Oct 25, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
hacktoberfest-2022 ✅ Ready (Inspect) Visit Preview Oct 25, 2022 at 10:46PM (UTC)

Copy link
Collaborator

@maadeval maadeval left a comment

Choose a reason for hiding this comment

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

Hola!! 👋 Los estilos en tus ficheros [...].css tienen que estar scopeados con tu nombre de usuario para que no entre en conflicto con otras soluciones o estilos de la página!

Te dejo un ejemplo:

#mvpsdvlpr .password-btn {
  ...
}

/* tambien puedes cambiar el nombre de las clases agregandole el nombre de usuario */
.mvpsdvlpr-password-btn {
...
}

Alternativa:
Otra opción es transformar esos ficheros a CSSModules ([...].module.css) para que el nombre de las clases se generen con un identificador único!

Por otro lado, revisa la preview del deploy https://hacktoberfest-2022-git-fork-mvpsdvlpr-main-midudev-pro.vercel.app/entry/mvpsdvlpr, vas a observar que tu solución no tiene la parte lógica de tu código. Para que astro pueda descargarse ese JavaScript debes agregar una nueva propiedad a tu componente en src/pages/entry/mvpsdvlpr/index.astro, tal que quede así:

---
import Layout from '@layout'
import Generatorpsswrd from '@components/mvpsdvlpr/Generatorpsswrd'
---

<Layout title='mvpsdvlpr'>
	<Generatorpsswrd client:load />
</Layout>

Te adjunto la documentación por si quieres saber más de esto 😊

Muchas gracias por tu solución!

@mvpsdvlpr
Copy link
Author

@maadeval grcias por los comentarios los revisare !

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.

2 participants