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 site and location to glossary #354

Merged
merged 4 commits into from
Oct 31, 2023
Merged

Conversation

tahliar
Copy link
Collaborator

@tahliar tahliar commented Oct 26, 2023

PR creator: Description

Added items for "site" and "location" in the glossary. I didn't include the non-constraint meaning of location, even though it's used in a few places, because it's just a regular non-technical word.

PR creator: Are there any relevant issues/feature requests?

  • jsc#DOCTEAM-108

PR creator: Which product versions do the changes apply to?

When opening a PR, check all versions of the documentation that your PR applies to.

  • SLE-HA 15
    • 15 next (current main, no backport necessary)
    • 15 SP5
    • 15 SP4
    • 15 SP3
    • 15 SP2
    • 15 SP1
  • SLE-HA 12
    • 12 SP5
    • 12 SP4

PR reviewer only: Have all backports been applied?

The doc team member merging your PR will take care of backporting to older documents.
When opening a PR, do not set the following check box.

  • all necessary backports are done

@tahliar
Copy link
Collaborator Author

tahliar commented Oct 26, 2023

Site:

image

Location:

image

@tahliar
Copy link
Collaborator Author

tahliar commented Oct 26, 2023

@zzhou1 @gao-yan What do you think? Is this okay or have I missed some nuance?

Copy link
Contributor

@gao-yan gao-yan left a comment

Choose a reason for hiding this comment

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

Very nice clarifications. Thanks, @tahliar !

Copy link
Contributor

@dariavladykina dariavladykina left a comment

Choose a reason for hiding this comment

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

LGTM.

@tahliar tahliar merged commit 4b8bd3a into main Oct 31, 2023
@tahliar tahliar deleted the DOCTEAM-108-site-vs-location branch October 31, 2023 04:34
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
tahliar added a commit that referenced this pull request Oct 31, 2023
* Clarify location terminology where appropriate

* Remove duplicate geo cluster entry from glossary

* Add site and location to glossary

jsc#DOCTEAM-108

* Geographic -> geographical to match the other uses in the guide
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