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]: Eval getting triggered twice on page switch in deployed mode #36237

Open
1 task done
sneha122 opened this issue Sep 11, 2024 · 0 comments · May be fixed by #36433
Open
1 task done

[Bug]: Eval getting triggered twice on page switch in deployed mode #36237

sneha122 opened this issue Sep 11, 2024 · 0 comments · May be fixed by #36433
Assignees
@rishabhrathod01
Labels
Bug Something isn't working Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Product Issues related to a specific integration Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage Query & JS Pod Issues related to the query & JS Pod Release

Comments

@sneha122
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Description

Recently we merged an improvement for consolidated view API's Action part, where we say that published actions will be fetched based on pageId instead of app Id. This way whenever we switch between pages, we call the consolidated api again to fetch actions belonging to the new page. This has caused an issue in the evaluations as mentioned here https://theappsmith.slack.com/archives/C024GUDM0LT/p1725943060293719?thread_ts=1725864964.488269&cid=C024GUDM0LT

Steps To Reproduce

  1. Create app and publish it
  2. In deployed mode, switch between different pages

Public Sample App

No response

Environment

Release

Severity

Medium (Frustrating UX)

Issue video log

No response

Version

NA
@sneha122 sneha122 added Bug Something isn't working Needs Triaging Needs attention from maintainers to triage labels Sep 11, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added Medium Issues that frustrate users due to poor UX Release labels Sep 11, 2024
@sneha122 sneha122 changed the title [Bug]: Issue in modules actions post merging actions based on pageId PR [Bug]: Eval getting triggered twice on page switch in deployed mode Sep 11, 2024
@rishabhrathod01 rishabhrathod01 added the Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. label Sep 16, 2024
@github-actions github-actions bot added Integrations Product Issues related to a specific integration Query & JS Pod Issues related to the query & JS Pod labels Sep 16, 2024
@rishabhrathod01 rishabhrathod01 linked a pull request Sep 19, 2024 that will close this issue
Open
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rishabhrathod01 rishabhrathod01

Labels
Bug Something isn't working Integrations Pod General Issues related to the Integrations Pod that don't fit into other tags. Integrations Product Issues related to a specific integration Medium Issues that frustrate users due to poor UX Needs Triaging Needs attention from maintainers to triage 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: remove redundant eval trigger appsmithorg/appsmith
4 participants
@Nikhil-Nandagopal @pranavkanade @rishabhrathod01 @sneha122