-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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: JS library not found in view mode after importing from Git #29297
Conversation
WalkthroughWalkthroughThe changes involve an update to the application import process from Git in Appsmith, where a new step to publish the application has been added. Additionally, there are updates to the test suite to reflect this new behavior, specifically by adding an assertion to check the size of a list named Changes
Assessment against linked issues
TipsChat with CodeRabbit Bot (
|
app/server/appsmith-server/src/test/java/com/appsmith/server/services/ce/GitServiceCETest.java
Outdated
Show resolved
Hide resolved
app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/GitServiceCEImpl.java
Show resolved
Hide resolved
app/server/appsmith-server/src/main/java/com/appsmith/server/services/ce/GitServiceCEImpl.java
Show resolved
Hide resolved
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/7096338202. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7096338202. |
@coderabbitai resolve |
/ok-to-test sha=43e1bc5 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/7100864566. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7100864566.
|
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7100864566.
|
/ok-to-test |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/7124096542. |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/7124096597. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7124096542. |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7124096597.
To know the list of identified flaky tests - Refer here |
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/7124096597. |
Description
If an application has a custom js library set, it's not available in view mode right after importing the application from git. User need to add a commit to make it available in Git. This PR performs a publish application operation right after git import so that any resource available in edit mode is also available in view mode.
PR fixes following issue(s)
Fixes #29278
Media
Type of change
Testing
How Has This Been Tested?
Test Plan
Issues raised during DP testing
Checklist:
Dev activity
QA activity:
Test Plan Approved
label after Cypress tests were reviewedTest Plan Approved
label after JUnit tests were reviewedSummary by CodeRabbit
New Features
Tests