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

Reviewing naming of Variable Sets #2360

Merged
merged 13 commits into from
Jun 26, 2024
Merged

Reviewing naming of Variable Sets #2360

merged 13 commits into from
Jun 26, 2024

Conversation

MarkSiedle
Copy link
Contributor

@MarkSiedle MarkSiedle commented Jun 25, 2024

In preparation for Vertical Navigation's release next week, we've been reviewing docs and discovered various combinations of the wording around "Variable Sets".

This PR reviews the various wording and standardises where appropriate, based on both the current UI and the new UI coming soon. We no longer refer to "Variable Sets" as "Library Variable Sets" or "Library Sets". They are just Variable Sets.

@steve-fenton-octopus
Copy link
Contributor

There are quite a few images in here with a missing opening (...

![]/docs/shared-content/octopus-recommendations/configure-project/images/projectconfiguration-deployareleasemanualintervention.png)

Should be:

![](/docs/shared-content/octopus-recommendations/configure-project/images/projectconfiguration-deployareleasemanualintervention.png)

Sending a commit to fix this and resolve some spelling issues.

@steve-fenton-octopus
Copy link
Contributor

I can sort out this merge for you tomorrow.

@MarkSiedle
Copy link
Contributor Author

Ah it was the image tags! I was so confused by the spell check errors. Thank you. I'm still reviewing some of these changes later today, but this is in a much nicer position now, thank you.

@MarkSiedle MarkSiedle marked this pull request as ready for review June 25, 2024 22:36
@MarkSiedle
Copy link
Contributor Author

Thanks @steve-fenton-octopus. This one is now ready for review please 🙏

Copy link
Contributor

@steve-fenton-octopus steve-fenton-octopus left a comment

Choose a reason for hiding this comment

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

LGTM!

I'll add a note to the "next edition feedback" of Exploring Octopus Deploy to also remove "library" and just use "variable set".

@MarkSiedle MarkSiedle merged commit debcbb4 into main Jun 26, 2024
4 checks passed
@MarkSiedle MarkSiedle deleted the fft/variable-sets branch June 26, 2024 22: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