Skip to content

Conversation

@1000TurquoisePogs
Copy link
Member

This enhancement allows you to put the following in your YAML

components:
  tn3270-ng2:
    enabled: false

If you did that prior, nothing much would happen. No errors, but no benefit.
Now, this is respected.

The logic is a bit silly:

  1. startup checks if the /plugins folder needs to be reset and add back all bundled and core plugins based on if core plugins are there
  2. extensions and non-bundled plugins are added or removed from /plugins folder depending upon "enabled" status of yaml
  3. NEW - bundled plugins are REMOVED if the YAML has them marked disabled

So, 1) adds them blindly, and 3) removes them conditionally. Kind of silly but I did not want to cause any backward compatibility issues or update the "defaults" folder.

Defaults are unchanged. This just allows you to customize which plugins are available.

In this PR I also spotted a bug about manifest.json not being handled.
I have never seen a manifest.json. They have always been manifest.yaml
But during v2.0 development, we said it would be possible and so this code was written but untested I guess.

Copy link
Contributor

@timgerstel timgerstel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@github-project-automation github-project-automation bot moved this from Ready for Review to Merge Ready in WebUI planning board Sep 29, 2025
Signed-off-by: 1000TurquoisePogs <[email protected]>
Copy link
Contributor

@timgerstel timgerstel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now properly searches Zowe share folder for components like the ones in system-apps

@1000TurquoisePogs 1000TurquoisePogs merged commit d946bad into v3.x/staging Oct 27, 2025
5 of 7 checks passed
@github-project-automation github-project-automation bot moved this from Merge Ready to Closed in WebUI planning board Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Closed

Development

Successfully merging this pull request may close these issues.

3 participants