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

README install assumes use of WordPress <5.6 and use of application-passwords plugin #837

Closed
1 task done
skorasaurus opened this issue Jan 28, 2022 · 0 comments · Fixed by #838
Closed
1 task done
Labels
type:enhancement New feature or request.

Comments

@skorasaurus
Copy link
Contributor

Is your enhancement related to a problem? Please describe.

I found the installation instructions in the README fairly confusing since I am using WordPress 5.8.x (Application Passwords has been integrated into core since 5.6). Even with the application-passwords plugin installed, the directions did not quite match the UI that is presented by WordPress.

Notably, the 'add external connection' screen by default just prompts you to enter the website's rest URL; there is no visible field for the application password.
image

Once you click the authorize connection, you're brought to the login screen of the main site. I eventually figured out that this requires the main site's user name and user's password (NOT the application password) which is discouraged from a security perspective.

I'll make a PR that hopefully more clearly explains how to use the application password

Designs

No response

Describe alternatives you've considered

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@skorasaurus skorasaurus added the type:enhancement New feature or request. label Jan 28, 2022
jeffpaul added a commit that referenced this issue Feb 24, 2022
clarify adding external connections instructions in README, fix #837
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement New feature or request.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant