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

chore: fix the navigation #614

Draft
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

Garfield-fr
Copy link
Contributor

When a resource is added or modified, it is not always returned to the correct page. The editor now uses the previous url to redirect.

@Garfield-fr Garfield-fr self-assigned this Apr 30, 2024
@Garfield-fr Garfield-fr force-pushed the zub-navigation-fix branch 2 times, most recently from 15442de to 92895f8 Compare April 30, 2024 08:51
Copy link
Contributor

@PascalRepond PascalRepond left a comment

Choose a reason for hiding this comment

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

This is not what was asked in rero/rero-ils#3628.

The problem that the back button/browser function sometimes takes us to an editor view, sometimes not is not fixed by this PR.

When a resource is added or modified, it is not always returned
to the correct page. The editor now uses the previous url to redirect.

* Closes rero/rero-ils#3628.

Co-Authored-by: Bertrand Zuchuat <[email protected]>
Copy link
Contributor

@PascalRepond PascalRepond left a comment

Choose a reason for hiding this comment

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

Not-blocking: doesn't work for library editor beacuse it is not formly/jsonschema based. But this is not important as it is used very little.

@Garfield-fr Garfield-fr marked this pull request as draft May 14, 2024 10:07
@PascalRepond
Copy link
Contributor

After discussion, we decided to wait on this PR because it has unintended effects on the current workflows for professionnals, for example:

  • Importing a document and saving it brings us straight back to the "import" window for this document
  • Creating a document from the main menu brings us back to the main menu but we usually want to see the document we just created.
  • Etc.

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.

"Back" button on certain resources opens the editor
3 participants