Skip to content

Add project dashboard page #2684

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

Merged
merged 17 commits into from
Apr 30, 2025

Conversation

Rose-Northey
Copy link
Contributor

@Rose-Northey Rose-Northey commented Apr 17, 2025

Description

Add project dashboard page which explains how releases are selected for display on the various views of the project dashboard.
[sc-98610]

After Change

The project dashboard page was added.
image

It is also linked in a number of places in the docs, especially when a user may want to understand which releases will be displayed on the dashboard.

@CLAassistant
Copy link

CLAassistant commented Apr 17, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@steve-fenton-octopus
Copy link
Contributor

I've sent this PR through the deployment channel to staging so you can check whether it looks how you want!

http://octopus-docs-staging.s3-website-us-east-1.amazonaws.com/docs/

Copy link
Contributor

@steve-fenton-octopus steve-fenton-octopus left a comment

Choose a reason for hiding this comment

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

Hey, there are three things on the to do list.

  1. Signing the CLA
  2. We use asterisk-emphasis in our markdown lint settings
  3. The frontmatter on the new page needs an update

Let me know if you need help with this.

@Rose-Northey Rose-Northey marked this pull request as draft April 28, 2025 03:11
@Rose-Northey Rose-Northey changed the title Add project dashboard page previous: Add project dashboard page Apr 28, 2025
@Rose-Northey Rose-Northey force-pushed the rosen/which-releases-are-selected-for-dashboard branch from 48dadf4 to 661b01f Compare April 28, 2025 04:10
@Rose-Northey Rose-Northey changed the title previous: Add project dashboard page Add project dashboard page Apr 28, 2025
@Rose-Northey Rose-Northey marked this pull request as ready for review April 29, 2025 01:19
Copy link
Contributor

@susanpann susanpann left a comment

Choose a reason for hiding this comment

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

Looks good! I see a few formatting changes for the better (removing excess spaces, whitespaces, etc), so let's keep those just for this pr. In src/pages/docs/administration/retention-policies/index.mdx there are a few figures where the width was removed - let's add those back in

Otherwise, great work breaking down the different views!

@steve-fenton-octopus
Copy link
Contributor

steve-fenton-octopus commented Apr 30, 2025

Looks good! I see a few formatting changes for the better (removing excess spaces, whitespaces, etc), so let's keep those just for this pr. In src/pages/docs/administration/retention-policies/index.mdx there are a few figures where the width was removed - let's add those back in

Otherwise, great work breaking down the different views!

Hey @susanpann the new docs site doesn't use those DocSync image widths so they have no effect.

Images can't get bigger than 764px as the layout is constrained. If we need smaller images, we could create addition classes for the figure element.

@Rose-Northey Rose-Northey merged commit 5e3853a into main Apr 30, 2025
4 checks passed
@Rose-Northey Rose-Northey deleted the rosen/which-releases-are-selected-for-dashboard branch April 30, 2025 21:42
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.

4 participants