-
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
fix: missing application id in comment threads API #14011
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@Parthvi12 Can you please test out the DP ? |
Unable to find test scripts. Please add necessary tests to the PR. |
/ok-to-test sha=5217a6e |
Unable to find test scripts. Please add necessary tests to the PR. |
5 similar comments
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Unable to find test scripts. Please add necessary tests to the PR. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/2368777917. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2368777917. Click to view performance test results
|
1 similar comment
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2368777917. Click to view performance test results
|
@arunvjn For existing app its working fine but I am seeing this toast message on creating new app |
/ok-to-test sha=53ddb8d |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/2370498136. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2370498136. Click to view performance test results
|
1 similar comment
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/2370498136. Click to view performance test results
|
Tested the dp and its working as expected @arunvjn |
Description
Fixes the error toast message that says "Please enter a valid applicationId".
This has to do with the comments component mounted and it the fetch threads API being called before the applicationId is retrieved. This PR delays the comments component mount in the application header.
Fixes #13489
Type of change
How Has This Been Tested?
Checklist:
Test coverage results 🧪
⚪ Total coverage has not changed