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

Search & Result Refactoring #320

Open
jonatanvicente opened this issue Apr 12, 2023 · 0 comments
Open

Search & Result Refactoring #320

jonatanvicente opened this issue Apr 12, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed My Environment My Environment features

Comments

@jonatanvicente
Copy link
Collaborator

jonatanvicente commented Apr 12, 2023

Es necesario efectuar algunas refactorizaciones de notable impacto:

Routing

  • Es necesario establecer una jerarquía correcta de rutas parent-child en los componentes MyEnvironmentPage y SearchAndResult.

my-environment-page.component.html

search-and-result.component.html

  • Refactorizar las subscriptions utilizando la nomenclatura: name$!: Subscription;
  • Añadir unsubscribe a todas las subscriptions en el método onDestroy
  • Debe capturarse la información necesaria proveniente del elemento padre mediante eventos
  • El método defineSearchType debe desaparecer
  • Es necesario eliminar los tipados any
@jonatanvicente jonatanvicente converted this from a draft issue Apr 12, 2023
@jonatanvicente jonatanvicente added help wanted Extra attention is needed My Environment My Environment features enhancement New feature or request labels Apr 12, 2023
@jonatanvicente jonatanvicente changed the title Search & Result refactoring Search & Result Refactoring Apr 12, 2023
@Almenara Almenara self-assigned this Apr 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed My Environment My Environment features
Development

No branches or pull requests

2 participants