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

add guide for import export feature #768

Merged
merged 5 commits into from
Jun 1, 2023
Merged

Conversation

owaincuvelier
Copy link
Contributor

quick guide on how to use the import export feature.

@owaincuvelier owaincuvelier requested review from rmccue and removed request for rmccue May 16, 2023 15:12
@owaincuvelier
Copy link
Contributor Author

just removing ryans review request, need to proof read first.

@owaincuvelier
Copy link
Contributor Author

quick guide on importing/exporting in the Altis Dashboard

Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

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

Looks like a lot but hopefully useful.

docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

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

I also noticed the screen web page you link to references a video that is hosted on a site Firefox (or possibly Cloudflare) regards as a "suspected phishing site". Probably worth finding another reference.

Update: The phishing warning has gone away now/

@owaincuvelier
Copy link
Contributor Author

@mikelittle I've made suggested changes, but left the screen link as is - I think you've run into a false positive perhaps.

Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

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

Sorry, it looks like a load more. But we are getting there now.

docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
docs/dashboard/import-export.md Outdated Show resolved Hide resolved
@owaincuvelier
Copy link
Contributor Author

@mikelittle thanks for reviewing again! I've just pushed the updates.

Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

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

One last minor change!

The only other thing I'd mention is that technically, the Markdown file should be wrapped at 80 characters. Given this is a new file, it would be good to format it correctly.
If you have a full environment, we have a lint command https://docs.altis-dxp.com/dev-tools/linting-your-documents/
But it's a good document. I'm not too worried about this last bit.


**Important** before running the search-replace it's a good idea to create a `screen` session. If you are unfamiliar with screen sessions, checkout [this helpful article](https://linuxize.com/post/how-to-use-linux-screen/#starting-linux-screen). As the [maximum CLI session time](https://docs.altis-dxp.com/cloud/dashboard/cli/#limitations) is 20 minutes, running the search-replace in a `screen` session avoids this limit.

If you’d like to learn more or are unfamiliar with WP CLI search-replace command, we recommended to familiarise yourself with these commands. Make use of the `--dry-run` flag before making permanent changes to the database. See the following article for more information: [https://developer.wordpress.org/cli/commands/search-replace](https://developer.wordpress.org/cli/commands/search-replace/).
Copy link
Contributor

Choose a reason for hiding this comment

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

"we recommended to familiarise yourself" > "we recommended you familiarise yourself"

Copy link
Contributor

@mikelittle mikelittle left a comment

Choose a reason for hiding this comment

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

Awesome! All good. Thanks for your patience.

@owaincuvelier owaincuvelier merged commit 70a8185 into master Jun 1, 2023
@owaincuvelier owaincuvelier deleted the import-export-guide branch June 1, 2023 13:31
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