Skip to content

fix: render fail on workspace dropdown in req group settings #6350

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

Merged
merged 3 commits into from
Aug 24, 2023

Conversation

filfreire
Copy link
Contributor

@filfreire filfreire commented Aug 22, 2023

changelog(Fixes): Fixed an issue where the workspace dropdown in the request folder settings would sometimes return an render fail error when picking through different workspaces to move/copy to.

Steps to reproduce:

  • checkout latest develop
  • make sure you have a handful of collections and design docs in your current project
  • in one of the collections/design docs go the the settings of a request folder
  • in the settings - on the workspace dropdown - try to click through a couple of workspace (e.g. picking on, re-opening the dropdown, pick another, reopen dropdown, ... repeat) until you get a render error

Copying the approach we do for the same dropdown in request settings was enough to fix the issue.

Pasted_Image_22_08_2023__09_57

@filfreire filfreire requested a review from a team August 22, 2023 09:08
@jackkav
Copy link
Contributor

jackkav commented Aug 22, 2023

We should merge the sidebar rewrite and retest this

@filfreire filfreire force-pushed the chore/fix-reqgroup-settings-workspace-dropdown branch from 04b0717 to 32eac0f Compare August 23, 2023 07:56
@filfreire filfreire force-pushed the chore/fix-reqgroup-settings-workspace-dropdown branch from 32eac0f to 2fe1d75 Compare August 23, 2023 08:29
@jackkav jackkav merged commit 6075f15 into develop Aug 24, 2023
@jackkav jackkav deleted the chore/fix-reqgroup-settings-workspace-dropdown branch August 24, 2023 08:42
jackkav added a commit to jackkav/insomnia that referenced this pull request Nov 24, 2023
* fix: render fail on workspace dropdown in req group settings

* match move/copy select implementation

* fix move request navigate

---------

Co-authored-by: jackkav <jackkav@gmail.com>
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.

2 participants