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

fix error when migrated layer does not have srid configured correctly… #1

Closed

Conversation

LaimeJesus
Copy link

@LaimeJesus LaimeJesus commented Aug 20, 2020

Este PR agrega los siguientes cambios

@cristiandley
Copy link

Si pueden linkear la tarea al PR por favor (dejar link en el PR de la tarea). Para mantener referencia con lo solicitado.

@LaimeJesus
Copy link
Author

cristiandley
cristiandley previously approved these changes Sep 7, 2020
@asassano
Copy link

asassano commented Sep 9, 2020

@LaimeJesus porque integra en prod-backup y no en develop?? esta rama es libre y la podes mergear vos??

@LaimeJesus LaimeJesus changed the base branch from prod-backup to develop September 11, 2020 20:05
@LaimeJesus LaimeJesus dismissed cristiandley’s stale review September 11, 2020 20:05

The base branch was changed.

@LaimeJesus LaimeJesus force-pushed the hotfix/fix-invalid-project-or-layer-is-missing-crs branch from dcab408 to 60cb720 Compare September 11, 2020 21:51
@LaimeJesus LaimeJesus added the bug Something isn't working label Sep 11, 2020
@cristiandley
Copy link

@LaimeJesus Por que hay un force push despues de la declaracion del Pull Request ?

Copy link

@asassano asassano left a comment

Choose a reason for hiding this comment

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

@LaimeJesus cuando puedas aclaremos esto #1 (comment) para saber si es crítico o no y @cristiandley veamos la aprobación así ya se integra en develop. Gracias

@LaimeJesus
Copy link
Author

@LaimeJesus Por que hay un force push despues de la declaracion del Pull Request ?
@cristiandley @asassano

Hay que desestimar este PR, este repositorio tiene un problema desde la creación de la rama develop.
Actualmente el código funcionando en CRESUD, PROD y CLON_PROD utiliza la versión de Geonode de la rama prod-backup, por lo tanto no tiene sentido mergear estos cambios a una rama que no sigue la misma versión, en este caso master(hay cambios en enero 2020 del equipo de Geonode) y develop que se desprende de la misma.

Por lo tanto para mantener la consistencia de versiones que estamos utilizando en los servidores mensionados deberíamos continuar usando la rama prod-backup o en su defecto la rama RELEASE que tiene implementados sus cambios.
Nosotros detectamos que la rama "más parecida" a lo que se encuentra funcionando en los servidores productivos es la rama 2.7.x.

En conclusión, tenemos que:

  • eliminar la rama develop
  • crear una nueva rama develop a partir de "prod-backup" o "2.7.x".
  • armar un nuevo PR hacia develop desde la rama hotfix/fix-invalid-project-or-layer-is-missing-crs
  • continuar usando las mismas versiones que en los servidores productivos

@cristiandley
Copy link

@LaimeJesus creen el nuevo PR y validamos ese.

@marcelorubini marcelorubini deleted the hotfix/fix-invalid-project-or-layer-is-missing-crs branch October 6, 2020 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants