Restore visit previous URL stored in session #249
Conversation
|
Con este cambio ya funcionaria mucho mejor de lo que tenemos en producción, sin embargo deberíamos solvetar también la gestión de errores, http://community.coopdevs.org/t/gestion-de-errores/190 antes de ponerlo en producción ya que si un usuario que está en varios Bancos de Tiempo intenta entrar a una oferta y esta no pertenece al BdT que detecta el sistema al hacer login (por defecto entras a un BdT y luego puedes cambiar a los otros de los que formas parte) da error y el usuario no puede seguir navegando. En el futuro resolveremos también este problema http://community.coopdevs.org/t/scope-todos-los-recursos-de-organization-bajo-organizations/25 Pero por ahora con este PR y la gestión de errores http://community.coopdevs.org/t/gestion-de-errores/190 podemos actualizar producción |
|
Yo creo que la tarea "scopes" resolveria muchos de estos problemas y es la mas prioritaria. Es cierto que es un refactor un poco chungo y requiere mas tiempo. |
enricostano
force-pushed
the
fix/restore-session-previous-url
branch
from
76ba8bc to
31e6900
Compare
|
@sseerrggii Can we close this PR? Is the issue really solved? |
|
Not really, but we can close it and some day organization scope will fix it |
Some time ago we introduced Devise restore previous URL after sign in with #148
Later we tried to fix some issue with ed87323 and 601aece
The result is that we removed that Devise functionality, but we don't have enough context to say why since the mentioned commits give no information about the issues.
Since users use posts links from emails they are always redirected to
/membersafter login instead of the post they where looking for.With this PR we re-introduce the functionality and we'll see if we'll encounter again those issue. 🙏