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

Update the website content #459

Merged
merged 1 commit into from
Jun 1, 2023
Merged

Update the website content #459

merged 1 commit into from
Jun 1, 2023

Conversation

ipanova
Copy link
Member

@ipanova ipanova commented May 16, 2023

[noissue]

@ipanova ipanova marked this pull request as draft May 16, 2023 18:27
ipanova
ipanova commented May 16, 2023
View reviewed changes
pulp-ha.md Outdated Show resolved Hide resolved
ggainey
ggainey requested changes May 26, 2023
View reviewed changes
about-pulp-3.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
get_involved.md Outdated Show resolved Hide resolved
installation-introduction.md Outdated Show resolved Hide resolved
migration.md Outdated Show resolved Hide resolved
migration.md Outdated Show resolved Hide resolved
@lubosmj
Copy link
Member

lubosmj commented May 26, 2023

@ipanova, would you mind reviewing the podman-compose section too? The first paragraph evokes non-readiness for the real installation and deployment. However, this is the recommended way to install Pulp. I am talking about this sentence:

Because this podman compose is new, so far it has received limited feedback and testing.

@ipanova ipanova force-pushed the no-pulp-installer branch 3 times, most recently from 531a7fa to 2a376a1 Compare May 29, 2023 16:36
@ipanova ipanova marked this pull request as ready for review May 29, 2023 16:37
lubosmj
lubosmj approved these changes May 29, 2023
View reviewed changes
Copy link
Member

@lubosmj lubosmj left a comment

Choose a reason for hiding this comment

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

Great overhaul! Do we need to attach the issue #458?

documentation.md Outdated
@@ -10,7 +10,6 @@ toc: false
## Pulpcore Documentation

* Latest: [docs.pulpproject.org](http://docs.pulpproject.org).
Copy link
Member

Choose a reason for hiding this comment

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

Would you mind changing "http" occurrences with "https"?

Copy link
Member

Choose a reason for hiding this comment

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

We can then burn another issue down: #94.

Copy link
Member Author

@ipanova ipanova May 29, 2023 •
edited

Choose a reason for hiding this comment

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

This issue is from year 2017. I updated links from posts from 3 years ago but did not go any further. If you're fine I will close out that issue this way, I do not see sense to update posts from 2011 where links are broken, or links that do not exist anymore http://pulp-python.readthedocs.io/en/3.0.0b1/, or links related to pulp2.

Copy link
Member

Choose a reason for hiding this comment

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

I had in mind updating the links we currently present to our users -- like the above one where this discussion happens; not the old ones.

Copy link
Member Author

Choose a reason for hiding this comment

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

That single reference was updated too.

ggainey
ggainey requested changes May 29, 2023
View reviewed changes
Copy link
Contributor

@ggainey ggainey left a comment

Choose a reason for hiding this comment

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

Minor wordsmithing - great work Ina!

content-plugins.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
help.md Outdated Show resolved Hide resolved
_data/topnav.yml Outdated Show resolved Hide resolved
This was referenced May 30, 2023
Open
Closed
ggainey
ggainey requested changes May 30, 2023
View reviewed changes
Copy link
Contributor

@ggainey ggainey left a comment

Choose a reason for hiding this comment

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

Almost there! :)

installation-introduction.md Outdated Show resolved Hide resolved
content-plugins.md Outdated Show resolved Hide resolved
@ipanova
Updated the website content
e82bd6b 
Removed references to pulp_installer as supported installation method.
Updated broken links, outdated content.
Update relevant http assets to be https based.

closes #pulp#458
closes pulp#94
@ipanova ipanova merged commit c325711 into pulp:gh-pages Jun 1, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lubosmj lubosmj lubosmj approved these changes

@ggainey ggainey ggainey approved these changes

@mikedep333 mikedep333 Awaiting requested review from mikedep333

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@ipanova @lubosmj @mikedep333 @ggainey