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.

Sign up for GitHub

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

Jump to bottom

Running commands remotely: On restrictive webhost #425

Merged
merged 3 commits into from
Dec 13, 2022
Merged

Running commands remotely: On restrictive webhost #425

merged 3 commits into from
Dec 13, 2022

Conversation

porg
Copy link
Contributor

@porg porg commented Dec 13, 2022

  • Added a 4th method which works on restrictive webhosts too:
    • Remote non-interactive shell resolves aliases and runs wp as alias with php.
  • Transformed the other 3 methods from free text to sections with a headline.
    • All headlines created during this edit now also produce nice HTML anchor links intended as permalinks - Internal jumplinks, which I already make use of. - And to guide externals visitors directly to their recommended method (chat, email, social media)
    • I tried to give the headlines a timeless and long lasting name as good as possible.
      • Please treat them as permalinks.
      • If wrong, then please correct them now, before they start their lifecycle as permalinks/deeplinks.
  • It took me 1.5 working days of research and trial and error until I finally found a way to run wp-cli remotely on my webhost.
    • Eventually was glad when I found a way. And I want to share this!
    • Also put quite some effort into this writeup.
    • I hope that this makes it to the website.
    • Please feel free to further correct / amend. You are the expert(s)!

@porg
Running commands remotely: On restrictive webhost
bd0526f 
- Added a 4th method which works on restrictive webhosts too:
  - Remote non-interactive shell resolves aliases and runs wp as alias with php.
- Transformed the other 3 methods from free text to sections with a headline.
  - All headlines created during this edit now also produce nice HTML anchor links intended as permalinks
    - Internal jumplinks, which I already make use of.
    - And to guide externals visitors directly to their recommended method (chat, email, social media)
  - I tried to give the headlines a timeless and long lasting name as good as possible.
    - Please treat them as permalinks.
    - If wrong, then please correct them now, before they start their lifecycle as permalinks/deeplinks.
- It took me 1.5 working days of research and trial and error until I finally found a way to run wp-cli remotely on my webhost.
  - Eventually was glad when I found a way. And I want to share this!
  - Also put quite some effort into this writeup.
  - I hope that this makes it to the website.
  - Please feel free to further correct / amend. You are the expert(s)!
@porg porg requested a review from a team as a code owner December 13, 2022 21:40
@porg porg mentioned this pull request Dec 13, 2022
Open
1 task
danielbachhuber
danielbachhuber approved these changes Dec 13, 2022
View reviewed changes
Copy link
Member

@danielbachhuber danielbachhuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, @porg — thank you!

@danielbachhuber danielbachhuber merged commit 0923806 into wp-cli:main Dec 13, 2022
@danielbachhuber danielbachhuber added this to the 2.8.0 milestone Dec 13, 2022
@porg
Copy link
Contributor Author

porg commented Dec 13, 2022

Please restore the extra spaces!

  • Yes these path style as in the headlines may seem a bit strange, but this is necessary for the anchor links, which serve as permalinks both for arriving and also internally as cross-links.
  • The real correct paths are then anyhow in the free text.

Apart from this

  • When will this go live on the website?

@danielbachhuber
Copy link
Member

@porg Sorry about that! Want to fix them in a follow-up PR?

I forget how often this syncs, but I think it's at least once a day.

@porg
Copy link
Contributor Author

porg commented Dec 13, 2022

  • I lost my git practise and would take long for this or worse screw up. I kindly ask you to please do this reverting.

  • Alternatively: If you know a solution to define a Headline and define the anchor link for it manually (in the Markdown file) which then makes it to the live HTML site, this would ofc be the "best of both worlds".

    • But on the other side, future editors may not be familiar with this special markup and discard it.
    • The current compromise is quite acceptable I'd say.

I am so proud that this my contribution makes it online 🙂

@danielbachhuber
Copy link
Member

@porg Sounds good! I've reverted the change: b401f12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielbachhuber danielbachhuber danielbachhuber approved these changes

Assignees
No one assigned
Labels
scope:documentation
Projects
None yet
Milestone
2.8.0
Development

Successfully merging this pull request may close these issues.
2 participants
@porg @danielbachhuber