Skip to content

wording changes #253

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
Open
wants to merge 31 commits into
base: main
Choose a base branch
from
Open

wording changes #253

wants to merge 31 commits into from

Conversation

phwecker
Copy link

@phwecker phwecker commented Jun 4, 2025

  • added reference to cost incurring of ESU purchase (esp. back-billing) in challenge and solution doc
  • added wording around migration strategy as the UI might not reflect the option shown in the screenshot (and refered to be select in the text)

@cthoenes cthoenes requested review from skiddder and ASchwarz1984 June 4, 2025 10:46
@cthoenes cthoenes self-assigned this Jun 4, 2025
@cthoenes cthoenes added the documentation Improvements or additions to documentation label Jun 4, 2025
@cthoenes
Copy link
Collaborator

@skiddder @ASchwarz1984
Would you do me the favor of checking this one? :)

@skidder maybe some of it is already part of #245 ?

ASchwarz1984
ASchwarz1984 approved these changes Jun 12, 2025
View reviewed changes
Copy link
Contributor

@ASchwarz1984 ASchwarz1984 left a comment

Choose a reason for hiding this comment

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

Looks good - thank you

skiddder
skiddder approved these changes Jun 12, 2025
View reviewed changes
Copy link
Contributor

@skiddder skiddder left a comment

Choose a reason for hiding this comment

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

only format changes and an additional warning about costs. Approved.

@cthoenes cthoenes requested review from cthoenes and starkilla2003 and removed request for cthoenes June 13, 2025 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@starkilla2003 starkilla2003 Awaiting requested review from starkilla2003

2 more reviewers

@ASchwarz1984 ASchwarz1984 ASchwarz1984 approved these changes

@skiddder skiddder skiddder approved these changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@cthoenes cthoenes

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
Arc for Server
Development

Successfully merging this pull request may close these issues.
4 participants
@phwecker @cthoenes @ASchwarz1984 @skiddder