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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Action redesign - Unable to see Pages in Move to Page dropdown when there are just two pages. #38117

Closed
1 task done
btsgh opened this issue Dec 12, 2024 · 0 comments · Fixed by #38328
Closed
1 task done

[Bug]: Action redesign - Unable to see Pages in Move to Page dropdown when there are just two pages. #38117

btsgh opened this issue Dec 12, 2024 · 0 comments · Fixed by #38328
Assignees
@alex-golovanov
Labels
Bug Something isn't working High This issue blocks a user from building or impacts a lot of users IDE Navigation Issues/feature requests related to IDE navigation, and context switching IDE Pod Issues that new developers face while exploring the IDE IDE Product Issues related to the IDE Product Integrations Product Issues related to a specific integration Needs Triaging Needs attention from maintainers to triage QA Pod Issues under the QA Pod QA Needs QA attention Query Forms Isuses related to the query forms Query & JS Pod Issues related to the query & JS Pod Release

Comments

@btsgh
Copy link
Collaborator

btsgh commented Dec 12, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Description

When there are two pages in an application, when I try to move a query from Page1 to Page2, in the Move to Page driopdown, it shows up as No pages, though Page2 exists to move to it

Steps To Reproduce

  1. Create a new app
  2. Create Page1 and Page2
  3. Add a query from any datasource to Page1
  4. Try to do a Move to Page action, to move query to Page2
  5. Move to Page dropdown has No pages - and we cannot select Page2

https://jam.dev/c/339b9683-6a6d-4847-85f8-1f8bc35391eb

Public Sample App

No response

Environment

Release

Severity

High (Blocker to building or releasing)

Issue video log

No response

Version

Cloud
@btsgh btsgh added Bug Something isn't working High This issue blocks a user from building or impacts a lot of users Release Needs Triaging Needs attention from maintainers to triage Query Forms Isuses related to the query forms IDE Navigation Issues/feature requests related to IDE navigation, and context switching labels Dec 12, 2024
@github-actions github-actions bot added IDE Product Issues related to the IDE Product Integrations Product Issues related to a specific integration IDE Pod Issues that new developers face while exploring the IDE Query & JS Pod Issues related to the query & JS Pod labels Dec 12, 2024
@alex-golovanov alex-golovanov mentioned this issue Dec 23, 2024
Merged
2 tasks
alex-golovanov added a commit that referenced this issue Dec 24, 2024
@alex-golovanov
fix: cannot see move to page option when there are exactly two pages …
16953b8 
…in the app (#38328)

## Description
Fixes an issue where move between pages in query context menu was not
possible.

Fixes [#38117](#38117)

/ok-to-test tags="@tag.IDE"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/12468165994>
> Commit: 1ef7d57
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=12468165994&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.IDE`
> Spec:
> <hr>Mon, 23 Dec 2024 14:47:50 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No
@appsmith-bot appsmith-bot added the QA Needs QA attention label Dec 24, 2024
@github-actions github-actions bot added the QA Pod Issues under the QA Pod label Dec 24, 2024
NandanAnantharamu pushed a commit that referenced this issue Dec 27, 2024
@alex-golovanov
fix: cannot see move to page option when there are exactly two pages …
73d9fbe 
…in the app (#38328)

## Description
Fixes an issue where move between pages in query context menu was not
possible.

Fixes [#38117](#38117)

/ok-to-test tags="@tag.IDE"

### 🔍 Cypress test results
<!-- This is an auto-generated comment: Cypress test results  -->
> [!TIP]
> 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
> Workflow run:
<https://github.com/appsmithorg/appsmith/actions/runs/12468165994>
> Commit: 1ef7d57
> <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=12468165994&attempt=1"
target="_blank">Cypress dashboard</a>.
> Tags: `@tag.IDE`
> Spec:
> <hr>Mon, 23 Dec 2024 14:47:50 UTC
<!-- end of auto-generated comment: Cypress test results  -->


## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [ ] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alex-golovanov alex-golovanov

Labels
Bug Something isn't working High This issue blocks a user from building or impacts a lot of users IDE Navigation Issues/feature requests related to IDE navigation, and context switching IDE Pod Issues that new developers face while exploring the IDE IDE Product Issues related to the IDE Product Integrations Product Issues related to a specific integration Needs Triaging Needs attention from maintainers to triage QA Pod Issues under the QA Pod QA Needs QA attention Query Forms Isuses related to the query forms Query & JS Pod Issues related to the query & JS Pod Release
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: cannot see move to page option when there are exactly two pages in the app appsmithorg/appsmith
4 participants
@alex-golovanov @appsmith-bot @albinAppsmith @btsgh