-
Notifications
You must be signed in to change notification settings - Fork 3
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
Added pingone_application export resource #77
Added pingone_application export resource #77
Conversation
PingDavidR
commented
Mar 15, 2024
- closes PDI-1541
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI we are waiting to merge these until the repo restructure pull request #71 is merged, so that we keep the git history clear.
internal/connector/pingone/resources/sso/pingone_application.go
Outdated
Show resolved
Hide resolved
"fmt" | ||
|
||
"github.com/pingidentity/pingctl/internal/connector" | ||
pingoneresourcescommon "github.com/pingidentity/pingctl/internal/connector/pingone/resources/common" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It doesn't look like this alias is necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so, if I remove the alias, what does the call on line 35 become?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just "common.[etc.]", since that is the non-aliased package name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm
Just need to merge in the latest |