Epic 472 port nginx one labs #681
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
documentation
✅ Deploy Preview will be available once build job completes!
|
y82
approved these changes
Jun 24, 2025
content/nginx-one/workshops/lab2/run-workshop-components-with-docker.md
Outdated
Show resolved
Hide resolved
content/nginx-one/workshops/lab3/explore-nginx-one-console-and-features.md
Outdated
Show resolved
Hide resolved
content/nginx-one/workshops/lab3/explore-nginx-one-console-and-features.md
Outdated
Show resolved
Hide resolved
static/workshops/nginx-one/lab2/nginx-oss/etc/nginx/conf.d/stub_status.conf
Outdated
Show resolved
Hide resolved
static/workshops/nginx-one/lab2/nginx-oss/etc/nginx/conf.d/tls-cars.example.com.conf
Outdated
Show resolved
Hide resolved
| @@ -0,0 +1,176 @@ | |||
| Apache License | |||
There was a problem hiding this comment.
Not sure about Apache License here, perhaps it should be clarified with SMEs:
- our license for docs is 2-clause BSD license
- we never published any licenses before
There was a problem hiding this comment.
@mjang -- Thoughts? I think you suggested I include this license when we first looked at the workshops; however, as Yar notes, the product docs don't have this kind of license.
* Ported lab 5
…entation into epic-472-port-nginx-one-labs
edits from tech review Co-authored-by: yar <y82@users.noreply.github.com>
This was
linked to
issues
Jun 25, 2025
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Write a clear and concise description that helps reviewers understand the purpose and impact of your changes. Use the
following format:
Problem: Give a brief overview of the problem or feature being addressed.
Solution: Explain the approach you took to implement the solution, highlighting any significant design decisions or
considerations.
Testing: Describe any testing that you did.
Please focus on (optional): If you any specific areas where you would like reviewers to focus their attention or provide
specific feedback, add them here.
If this PR addresses an issue on GitHub, ensure that you link to it here:
Closes #ISSUE
Checklist
Before merging a pull request, run through this checklist and mark each as complete.
README.mdandCHANGELOG.mdFootnotes
Potentially sensitive changes include anything involving code, personally identify information (PII), live URLs or significant amounts of new or revised documentation. Please refer to our style guide for guidance about placeholder content. ↩