Skip to content

Conversation

@fhennig
Copy link
Contributor

@fhennig fhennig commented May 27, 2024

also replaces the roles and role groups page - it was the first page I wrote and very wordy, and it fits nicely into the Stacklet page, as a Stacklet is mostly made up of roles.

I then went into all the concept pages and adjusted some text to mention the Stacklet in more places, as well as adjusting formatting to our style guide.

closes #541. The new page mentions that naming conflicts can occur.

@netlify
Copy link

netlify bot commented May 27, 2024

Deploy Preview for stackable-docs ready!

Name Link
🔨 Latest commit 51ba5a7
🔍 Latest deploy log https://app.netlify.com/sites/stackable-docs/deploys/6672d0af73fd3c00088cce5e
😎 Deploy Preview https://deploy-preview-613--stackable-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@fhennig fhennig marked this pull request as ready for review May 28, 2024 08:36
@fhennig fhennig self-assigned this May 28, 2024
@fhennig fhennig requested a review from razvan May 28, 2024 08:36
NickLarsenNZ
NickLarsenNZ previously approved these changes May 29, 2024
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

Super nit-picky, so approving and leaving it up to you whether you accept some of the suggestions.

Some stuff I commented on was just reformatted from what was already there, so not necessary to change in this PR.

Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
fhennig and others added 14 commits May 29, 2024 14:51
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
fhennig and others added 19 commits May 29, 2024 15:04
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
Co-authored-by: Nick <NickLarsenNZ@users.noreply.github.com>
fhennig and others added 2 commits June 12, 2024 12:25
@fhennig fhennig requested a review from NickLarsenNZ June 12, 2024 10:25
@fhennig fhennig enabled auto-merge June 19, 2024 12:36
Copy link
Member

@NickLarsenNZ NickLarsenNZ left a comment

Choose a reason for hiding this comment

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

LGTM

@fhennig fhennig added this pull request to the merge queue Jun 24, 2024
Merged via the queue into main with commit 61aea6c Jun 24, 2024
@fhennig fhennig deleted the add-stacklet-concepts-page branch June 24, 2024 07:10
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.

Add a concepts page for 'Stacklet'

4 participants