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

4.0 Remove Credentials option for GitHub as it's been phased out. #320

Merged
merged 6 commits into from
May 2, 2021

Conversation

particthistle
Copy link

@particthistle particthistle commented Mar 20, 2021

Pull Request for Issue #319.

Summary of Changes

Joomla 4 patch tester:

  • Edited config.xml to remove option from the dropdown and configuration fields
  • Removed language strings relating to the dropdown value and fields from en-GB

Testing Instructions

Go to Options for Joomla Patch Tester
Click on the GitHub Authentication Tab
When applied, Credentials option will be unavailable.

Before

Credentials Option is available
image

After

Credentials Option is unavailable.
image

As basic authentication no longer can be set, basic authentication should be removed.
Username and password for Github no longer able to be supplied in credentials, so can no longer be used by helper.
Modify string to remove credentials reference.
Update COM_PATCHTESTER_NO_CREDENTIALS to reference token not credentials.
@particthistle particthistle changed the title Remove Credentials option for GitHub as it's been phased out. 4.0 Remove Credentials option for GitHub as it's been phased out. Mar 20, 2021
@roland-d roland-d merged commit cfa82b4 into joomla-extensions:master May 2, 2021
@roland-d
Copy link
Collaborator

roland-d commented May 2, 2021

@particthistle Thank you.

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