Update readme for v3, and add ivy desktop plugin ID #316
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…d new desktop plugin to defaults Signed-off-by: 1000TurquoisePogs <sgrady@rocketsoftware.com>
| For the initial setup, the default authentication is the "trivial authentication" plugin, which allows login to the App Server without valid credentials. At the end of this guide, you can customize the environment to switch to a secure authentication plugin instead, such as the ZSS authentication plugin, covered in [Section 7](#7-adding-zss-to-the-environment). This plugin works with the [ZSS mock server](https://github.com/zowe/zss/tree/v2.x/staging/mock) as well. | ||
| If you wish to test against the v2.x desktop, you can include it via | ||
| ``` | ||
| git clone --branch v2.x/master git@github.com:zowe/zlux-app-manager.git zlux-app-manager-v2 |
There was a problem hiding this comment.
This implies V2 Desktop is solely testable by switching to v2 branch of zlux-app-manager which might be true now but may not be true for the whole V3 branch's lifecycle
There was a problem hiding this comment.
Prob best to recommend all V2 branches
| if not exist "%ZWE_zowe_workspaceDirectory%\app-server\plugins\org.zowe.zlux.json" ( | ||
| cd ..\lib | ||
| !NODE_BIN! initInstance.js | ||
| cd ..\bin | ||
| ) |
There was a problem hiding this comment.
This new script gives me an error. See our Slack group
This was referenced Aug 21, 2024
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR allows for both the v2 and v3 desktops to be used within a single zlux-app-server dev environment.
I adjusted the README to state how, but I also noticed the README needed other adjustments for v3.
Also, I noticed that app-server.bat did not match the behavior of start.sh, and adjusted that too to help windows users.
As I am not a windows user, I request testing to see if you can successfully start the app server using the readme instructions and app-server.bat