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

feat(pool): add password to fetched scratch org from pool #135

Merged

Conversation

thvd
Copy link

@thvd thvd commented Nov 5, 2024

This came as a proposal after getting stuck with this: (posted on the Slack channel)

Is there a reason why sfp doesn't persist the password in sf cli auth config after fetching a scratch org from the pool?
The reason I why I ask this is because Provar does generate a new password when it runs thus overriding the password as filled in the prod org Password__c field.

Reference to Provar source code: https://github.com/ProvarTesting/provardx/blob/development/src/utilities/ProvarDXUtility.ts#L112

I am thinking of figuring this out myself and providing a PR to sfp, what are your thoughts on this change?

@thvd thvd marked this pull request as ready for review November 5, 2024 09:20
@dieffrei
Copy link

dieffrei commented Nov 6, 2024

@thvd thank you for your contribution!

@dieffrei dieffrei self-requested a review November 6, 2024 17:48
@dieffrei dieffrei merged commit 299859b into flxbl-io:main Nov 6, 2024
2 checks passed
@thvd thvd deleted the feat/add-password-to-fetched-scratch-org branch November 7, 2024 06:36
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.

3 participants