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.

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

AAP-23926 Remove content from EDA GS guide that points customers to c… #1354

Merged
merged 1 commit into from
Jun 4, 2024
Merged

AAP-23926 Remove content from EDA GS guide that points customers to c… #1354

merged 1 commit into from
Jun 4, 2024

Conversation

jself-sudoku
Copy link
Contributor

@jself-sudoku jself-sudoku commented May 16, 2024 •
edited

Work that resolves AAP-23926, per SME:

  • Remove content that points customer to the EDA deployment using an internal database since it no longer exists and is not widely supported. Only point to Chapter 3 in the Installation Guide.
  • Also, remove links to two other scenarios that have been removed from the installation guide, per direction from support and products (guidance from Mike Silmser).

@jself-sudoku jself-sudoku added Ready for review This PR needs review and approval by members of the docs team Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch 2.4 Content applies to AAP 2.4 and removed Ready for review This PR needs review and approval by members of the docs team labels May 16, 2024
msilmser
msilmser approved these changes Jun 4, 2024
View reviewed changes
Copy link

@msilmser msilmser left a comment

Choose a reason for hiding this comment

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

lgtm

@jself-sudoku
Copy link
Contributor Author

Thanks, @msilmser.

@jself-sudoku jself-sudoku added the Technical review complete Content has been reviewed for technical accuracy and completeness label Jun 4, 2024
hherbly
hherbly approved these changes Jun 4, 2024
View reviewed changes
Copy link
Collaborator

@hherbly hherbly left a comment

Choose a reason for hiding this comment

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

LGTM

@hherbly hherbly merged commit 67e3a60 into ansible:main Jun 4, 2024
jself-sudoku added a commit to jself-sudoku/aap-docs that referenced this pull request Jun 4, 2024
@jself-sudoku
AAP-23926 Remove content from EDA GS guide that points customers to c…
0a68a86 
…ontent that no longer exists (ansible#1354)
@jself-sudoku jself-sudoku mentioned this pull request Jun 4, 2024
Merged
hherbly pushed a commit that referenced this pull request Jun 4, 2024
@jself-sudoku
AAP-23926 Remove content from EDA GS guide that points customers to c…
f7c596e 
…ontent that no longer exists (#1354) (#1397)
@jself-sudoku jself-sudoku added Backported to 2.4 Changes have been applied to the AAP 2.4 branch Backports confirmed Backports to release branches have been verified and removed Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch labels Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msilmser msilmser msilmser approved these changes

@hherbly hherbly hherbly approved these changes

Assignees
No one assigned
Labels
2.4 Content applies to AAP 2.4 Backported to 2.4 Changes have been applied to the AAP 2.4 branch Backports confirmed Backports to release branches have been verified Technical review complete Content has been reviewed for technical accuracy and completeness
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jself-sudoku @msilmser @hherbly