-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
feat: Datasource autosave improvements #17649
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Unable to find test scripts. Please add necessary tests to the PR. |
Deployment failed with the following error:
|
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=a335614 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cypress looks good!
/ok-to-test sha=13b93b5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3562934881. |
/ok-to-test sha=331ca85 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3565334955. |
/ok-to-test sha=3a84755 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3566206295. |
/perf-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3572922920. |
/perf-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3574437792. |
Description
In Appsmith we try to autosave everywhere. Today when a user clicks on a new datasource, it creates new named resource even though the user might have just been exploring.
Fixes #4381
Type of change
How Has This Been Tested?
Checklist: