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

Wc 129 dataset form redirect fix #1006

Open
wants to merge 14 commits into
base: task/digital-rocks
Choose a base branch
from

Conversation

van-go
Copy link
Contributor

@van-go van-go commented Nov 12, 2024

Overview

Dataset form redirect fix -- If we edit a dataset using the Edit Data button from within a dataset then the redirect navigates the user back one directory. The user should stay in the same directory where they clicked edit from.

Related

Changes

Update the reloadPath logic: Ensure reloadPath only replaces the file name when needed and keeps the full path intact if the directory has not changed.
Update reloadPage function: Modify reloadPage to avoid stripping the last directory when constructing projectUrl.

Testing

  1. Project > Select a Project > Open Sample Data > "Edit Data" > Make a change, update sample. Notice when the modal closes, you are still at the same page.

UI

Notes

van-go and others added 8 commits November 5, 2024 15:39
…projects (#998)

* add toolbar button and modal to display the tree for DRP projects

* scroll modal when it overflows the screen size
* Publication flow

* Refactored code, improved request pub flow

* remove incorrect return statement

* added versioning support + bug fixes

* front end linting

* bug fixes with trash, prevent copying of trashed files

* settings changes, review system only visible to reviewers

* refactor to use project tree to get samples
- Added citation box and citation modal for publications
- Removed Last Modified column on publication listing
- Changed Created to Published Date
- Added data view modal to view related project information
@van-go van-go requested review from taoteg and shayanaijaz November 12, 2024 19:45
@van-go van-go marked this pull request as ready for review November 12, 2024 19:59
Copy link
Contributor

@shayanaijaz shayanaijaz left a comment

Choose a reason for hiding this comment

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

I seem to be running into an issue. Whenever I edit a dataset using Edit Data it appends the new name to the url and redirects to an invalid location. Can you confirm?

@van-go van-go requested a review from shayanaijaz December 4, 2024 17:53
Copy link
Contributor

@shayanaijaz shayanaijaz left a comment

Choose a reason for hiding this comment

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

When I edit a dataset using the Edit Data button and change the name of the dataset, then the new name gets appended to the current url which leads to an invalid location. Other than that, everything else is working

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.

3 participants