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

Pd 1356 update apps UI ref content #2972

Merged
merged 17 commits into from
Aug 19, 2024
Merged

Conversation

micjohnson777
Copy link
Contributor

This PR updates screenshots and all UI ref related content in the Apps/ _index.md files in the UI Ref and Tutorial sections.

This PR depends on the PR for PD1352 merging to fix all the content in the tutorials/apps/_index.md file to avoid having to duplicate that effort in this PR. Marking this draft until that PR merges.

Thanks for contributing to TrueNAS documentation! By opening a Pull Request, you're acknowledging that your changes will be distributed under the Creative Commons 4.0 license.

This commit updates the UI ref content for the application common screens, removes content from the Apps tutorials _index.md file and creates a new snippet, then adds that snippet to both the UI ref and tutorial index files. It updates some of the UI images.  More UI changes are expected in nightly BETA.1 builds so screenshot commits are planned for this PR.
This commit updates a few screenshots.
PR for PD-1352 needs to merge into Master before fixing the tutorials/_index.md content.
@micjohnson777 micjohnson777 requested a review from a team as a code owner August 12, 2024 17:31
@micjohnson777 micjohnson777 marked this pull request as draft August 12, 2024 17:31
This commit changes content and screenshots based on the changes added to the BETA.1-INTERNAL-1 build.
Charts train is now just Stable.
Settings menu has Manage Container Images restored as an option.
Edit Catalog screen shows checkboxes for the trains now. It is not showing the enterprise train yet, shows as test for now. Expected changes in RC1 to add enterprise train.
@micjohnson777 micjohnson777 marked this pull request as ready for review August 13, 2024 14:44
@micjohnson777
Copy link
Contributor Author

Made updates based on what is in the BETA.1 build. More changes expected in RC1 but that should not hold up this PR. The PD-1352 PR MUST merge before this PR!

Mrt134
Mrt134 previously requested changes Aug 15, 2024
Copy link
Collaborator

@Mrt134 Mrt134 left a comment

Choose a reason for hiding this comment

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

Conflicts need to be resolved

@micjohnson777
Copy link
Contributor Author

Conflicts resolved.

@micjohnson777 micjohnson777 added Technical Writer Review (MJ) A technical writer is reviewing the PR. and removed Technical Writer Review (MJ) A technical writer is reviewing the PR. labels Aug 15, 2024
@Mrt134 Mrt134 self-requested a review August 15, 2024 18:47
@Mrt134
Copy link
Collaborator

Mrt134 commented Aug 15, 2024

Build is failing, there's a broken reference that needs to be addressed

delete relref to deleted catalog file.
@micjohnson777
Copy link
Contributor Author

removed link to deleted catalog file.

@Mrt134 Mrt134 dismissed their stale review August 15, 2024 19:50

stale

@Mrt134 Mrt134 added the Review (TM) A technical writer is reviewing the PR. label Aug 15, 2024
@DjP-iX DjP-iX added Technical Writer Review (DP) A technical writer is reviewing the PR. and removed Review (TM) A technical writer is reviewing the PR. labels Aug 16, 2024
DjP-iX
DjP-iX previously requested changes Aug 16, 2024
content/SCALE/SCALETutorials/Apps/UsingTrains.md Outdated Show resolved Hide resolved
content/SCALE/SCALETutorials/Apps/UsingTrains.md Outdated Show resolved Hide resolved
content/SCALE/SCALEUIReference/Apps/_index.md Outdated Show resolved Hide resolved
content/SCALE/SCALEUIReference/Apps/_index.md Show resolved Hide resolved
content/SCALE/SCALEUIReference/Apps/_index.md Show resolved Hide resolved
content/SCALE/SCALEUIReference/Apps/_index.md Show resolved Hide resolved
content/SCALE/SCALEUIReference/Apps/_index.md Show resolved Hide resolved
Copy link
Contributor Author

@micjohnson777 micjohnson777 left a comment

Choose a reason for hiding this comment

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

I'm working on the other late UI changes in PD-1373 PR

@tonyriv3 tonyriv3 dismissed DjP-iX’s stale review August 19, 2024 16:39

Suggestions implemented.

@tonyriv3 tonyriv3 merged commit 642a7e3 into master Aug 19, 2024
3 checks passed
@tonyriv3 tonyriv3 deleted the PD-1356-Update-Apps-UI-Ref-content branch August 19, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Technical Writer Review (DP) A technical writer is reviewing the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants