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

Add Release Notes for Foreman 3.9 #2629

Merged
merged 1 commit into from Dec 13, 2023
Merged

Conversation

Griffin-Sullivan
Copy link
Contributor

Please cherry-pick my commits into:

  • Foreman 3.9/Katello 4.11 (planned Satellite 6.15)
  • Foreman 3.8/Katello 4.10
  • Foreman 3.7/Katello 4.9 (Satellite 6.14)
  • Foreman 3.6/Katello 4.8
  • Foreman 3.5/Katello 4.7 (Satellite 6.13; orcharhino 6.6)
  • Foreman 3.4/Katello 4.6 (EL8 only)
  • Foreman 3.3/Katello 4.5 on EL7 & EL8 (Satellite 6.12 on EL8 only; orcharhino 6.4/6.5 on EL8 only)
  • Foreman 3.2/Katello 4.4 on EL7 & EL8
  • Foreman 3.1/Katello 4.3 on EL7 & EL8 (Satellite 6.11 EL7/8; orcharhino 6.3 on EL7/8)
  • We do not accept PRs for Foreman older than 3.1.

@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author and removed Not yet reviewed labels Dec 7, 2023
@pr-processor pr-processor bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Dec 7, 2023
ekohl
ekohl previously requested changes Dec 7, 2023
guides/doc-Release_Notes/topics/foreman-3.9.0.adoc Outdated Show resolved Hide resolved
Comment on lines 7 to 9
=== New All Hosts page features

The All Hosts page has been revamped to include features like Host Create and Register buttons, bulk host deletion, and faster links to each host. Users will now have the ability to do many CRUD tasks from the All Hosts page instead of having to go to other pages.
Copy link
Member

Choose a reason for hiding this comment

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

I think this is about the new experimental All Hosts page. AFAIK the existing All Hosts already has these.

cc @jeremylenz

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, this is about the new page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So should I remove this from the headline features? Or do I need to reword this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
=== New All Hosts page features
The All Hosts page has been revamped to include features like Host Create and Register buttons, bulk host deletion, and faster links to each host. Users will now have the ability to do many CRUD tasks from the All Hosts page instead of having to go to other pages.
=== New All Hosts page features
The new All Hosts page has been revamped to include features like Host Create and Register buttons, bulk host deletion, and faster links to each host. Users will now have the ability to do many CRUD tasks from the new All Hosts page instead of having to go to other pages.

Copy link
Member

Choose a reason for hiding this comment

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

Isn't this the first version to even introduce the new hosts page? Shouldn't it explain that it was introduced as a new experimental feature, intended to replace the existing page?

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, I thought it originally got into 3.8 for some reason!

Yes, then we should call it out as a new feature. Should call out that

  • The new page will show only if you set 'Show new host overview page' in Settings.
  • Initially, it will include features like ... < all the text we had here before >
  • In future releases, more features will be added to the new page, and the old page will eventually be retired.

Copy link
Member

Choose a reason for hiding this comment

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

@jeremylenz if you're up for it, you can also submit your own PR with the headline feature description. As one of its primary authors you probably know best.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Whatever works for me. @jeremylenz let me know what you'd like to do.

@pr-processor pr-processor bot added Waiting on contributor Requires an action from the author and removed Needs re-review labels Dec 7, 2023
@pr-processor pr-processor bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Dec 8, 2023
@evgeni evgeni merged commit 333c49a into theforeman:3.9 Dec 13, 2023
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants