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.

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

3 participants