Skip to content
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

Refactor ASP handling to use current iASP name and fetch additional information #2339

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

worksofliam
Copy link
Contributor

@worksofliam worksofliam commented Nov 9, 2024

Update the handling of ASPs to utilize the current iASP name instead of the deprecated sourceASP property, and enhance the fetching of ASP information by RDB name.

This means that Code for IBM i will always use the ASP that is defined as part of their user profile / job description.

The downside about this PR is I actually have no systems to test it on. So I kind of buggered up there.

Checklist

  • We need test cases for the new ASP methods in the IBMi class
  • Ensure that when an ASP is set that the user can browse and open members in *SYSBAS when they have an ASP configured
    • Object browser checked
    • Members checked
  • have tested my change
  • updated relevant documentation

@worksofliam
Copy link
Contributor Author

Ping @sebjulliand - please take a look at this in the next couple of weeks if you can.

Copy link
Contributor

github-actions bot commented Jan 13, 2025

👋 A new build is available for this PR based on 1953ac7.

src/api/IBMi.ts Outdated Show resolved Hide resolved
sebjulliand and others added 5 commits January 15, 2025 14:00
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Signed-off-by: Seb Julliand <[email protected]>
Co-authored-by: Sébastien Julliand <[email protected]>
Co-authored-by: Sébastien Julliand <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants