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

[Task]: Persist dependency map of a page in backend #33030

Closed
6 tasks done
dvj1988 opened this issue Apr 29, 2024 · 2 comments · Fixed by #33296
Closed
6 tasks done

[Task]: Persist dependency map of a page in backend #33030

dvj1988 opened this issue Apr 29, 2024 · 2 comments · Fixed by #33296
Assignees
Labels
Task A simple Todo Templates Pod Issues related to Templates Templates Issues related to templates

Comments

@dvj1988
Copy link
Contributor

dvj1988 commented Apr 29, 2024

The dependencyMap consumes considerable computation on the frontend. We would like to persist that in the backend for every page in an application.

For details on the signature and payload of the api calls please refer this document

Tasks

@dvj1988 dvj1988 added the Performance Pod All things related to Appsmith performance label Apr 29, 2024
@Nikhil-Nandagopal Nikhil-Nandagopal added the Task A simple Todo label Apr 29, 2024
@github-actions github-actions bot removed the Performance Pod All things related to Appsmith performance label Apr 29, 2024
@dvj1988
Copy link
Contributor Author

dvj1988 commented Apr 29, 2024

cc: @rahulbarwal @rajatagrawal

@dvj1988 dvj1988 changed the title [Task]: Add lastUpdateTime of page in 3 APIs [Task]: Persist dependency map of a page in backend May 2, 2024
@AnaghHegde AnaghHegde self-assigned this May 8, 2024
@AnaghHegde AnaghHegde added the Templates Pod Issues related to Templates label May 8, 2024
@github-actions github-actions bot removed the Templates Pod Issues related to Templates label May 8, 2024
@AnaghHegde AnaghHegde added the Templates Issues related to templates label May 8, 2024
@github-actions github-actions bot added the Templates Pod Issues related to Templates label May 8, 2024
@AnaghHegde
Copy link
Member

Adding more story points due to last movement review comment which requires change in the flow.

AnaghHegde added a commit that referenced this issue May 31, 2024
## Description
The dependencyMap consumes considerable computation on the frontend. We
would like to persist that in the backend for every page in an
application.

For details on the signature and payload of the api calls please refer
[this
document](https://www.notion.so/appsmith/Solution-design-dependencyMap-0c031675f13140f7a433c731b3e45470?pvs=4#1ee6692b74804422a80cba86cc9ba87f)
Slack thread -
https://theappsmith.slack.com/archives/C024GUDM0LT/p1714100372372069

Fixes #33030

## Automation

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

### 🔍 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/9253166755>
> Commit: 03e9f01
> Cypress dashboard url: <a
href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=9253166755&attempt=1"
target="_blank">Click here!</a>

<!-- end of auto-generated comment: Cypress test results  -->














## Communication
Should the DevRel and Marketing teams inform users about this change?
- [ ] Yes
- [x] No
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Task A simple Todo Templates Pod Issues related to Templates Templates Issues related to templates
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants