caverav / auditforge

AuditForge is a pentest reporting application making it simple and easy to write your findings and generate a customizable report.
https://auditforge.feriadesoftware.cl
MIT License
1 stars 0 forks source link

Avance: Data pareja1 #54

Closed jllanosg closed 2 weeks ago

jllanosg commented 2 weeks ago

Descripción

Agrega los avances para la sección data de la app.

En específico, agrega las siguientes funcionalidades:

El arreglo de estos queda pendiente, se hace este PR para tener una aproximación al resultado final, con el fin de poder mostrar resultados en el sprint review 1. La corección de los errores del linter quedó registrada en el issue #52.

Motivación y Contexto

Migración de frontend.

¿Cómo ha sido probado?

Capturas de pantalla (si es apropiado):

Tipos de cambios

Lista de verificación:

caverav commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

jllanosg commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

caverav commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

jllanosg commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

Estoy de acuerdo en lo primero, pero el tema de custom data es que ya tiene avances hechos, esta rama sale de data-pareja1 entonces no se qué pasaría con la misma si se borra la de origen

caverav commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

Estoy de acuerdo en lo primero, pero el tema de custom data es que ya tiene avances hechos, esta rama sale de data-pareja1 entonces no se qué pasaría con la misma si se borra la de origen

Si tiene cambios hechos, entonces al mergear se mergearán esos cambios, por ende no se perderán, y al hacer una nueva rama desde development, estarán esos cambios

jllanosg commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

Estoy de acuerdo en lo primero, pero el tema de custom data es que ya tiene avances hechos, esta rama sale de data-pareja1 entonces no se qué pasaría con la misma si se borra la de origen

Si tiene cambios hechos, entonces al mergear se mergearán esos cambios, por ende no se perderán, y al hacer una nueva rama desde development, estarán esos cambios

Los cambios no están en data-pareja1, solo en data-pareja1-custom-data. Sugieres que mergee primero data-pareja1-custom-data hacia data-pareja1?

caverav commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

Estoy de acuerdo en lo primero, pero el tema de custom data es que ya tiene avances hechos, esta rama sale de data-pareja1 entonces no se qué pasaría con la misma si se borra la de origen

Si tiene cambios hechos, entonces al mergear se mergearán esos cambios, por ende no se perderán, y al hacer una nueva rama desde development, estarán esos cambios

Los cambios no están en data-pareja1, solo en data-pareja1-custom-data. Sugieres que mergee primero data-pareja1-custom-data hacia data-pareja1?

Entonces ya está la nueva rama pues, para qué quieres data-pareja1?

jllanosg commented 2 weeks ago

Por qué no borrar la rama al mergear? Si hay que agregar una feature o fixear algo se hace una rama feature/ o bugfix/, o hay alguna razón para no seguir ese estándar?

Por lo que hablamos el otro dia (pushear estos cambios parcialmente para tenerlos juntos y poder mostrarlos); se seguirá trabajando en data-pareja1 para eliminar los problemas de eslint, y por otro lado la rama custom-data que está en un estado muy prematuro sigue a data-pareja1

Justamente el solucionar problemas de ESLint puede ir en una nueva rama, al igual que la feature de custom-data, es una forma de organizar las cosas, así además se puede hacer PR de una si ya está lista, y no tener que incluir cambios de la otra rama que tal vez ni estén completos

Estoy de acuerdo en lo primero, pero el tema de custom data es que ya tiene avances hechos, esta rama sale de data-pareja1 entonces no se qué pasaría con la misma si se borra la de origen

Si tiene cambios hechos, entonces al mergear se mergearán esos cambios, por ende no se perderán, y al hacer una nueva rama desde development, estarán esos cambios

Los cambios no están en data-pareja1, solo en data-pareja1-custom-data. Sugieres que mergee primero data-pareja1-custom-data hacia data-pareja1?

Entonces ya está la nueva rama pues, para qué quieres data-pareja1?

Nono, me expliqué mal, quiero decir que data-pareja1 se puede borrar pero no sé que pasaría con data-pareja-1-custom-data, simplemente queda en mergearla a dev?

caverav commented 2 weeks ago

Yep