Skip to content

[Docs] Update sharing-a-workspace, add promoting-a-design tutorial, and add kanvas/getting-started narrative callout#988

Merged
leecalcote merged 3 commits intomasterfrom
feat/narrative-976-kanvas-gs
Apr 29, 2026
Merged

[Docs] Update sharing-a-workspace, add promoting-a-design tutorial, and add kanvas/getting-started narrative callout#988
leecalcote merged 3 commits intomasterfrom
feat/narrative-976-kanvas-gs

Conversation

@leecalcote
Copy link
Copy Markdown
Member

@leecalcote leecalcote commented Apr 29, 2026

Summary

Depends on #979 (now merged) for the /cloud/about link target.

Note: These three commits landed on the same branch due to agent branch-context confusion during parallel execution. Content is correct and hugo build is clean.

@leecalcote leecalcote added help wanted Extra attention is needed issue/willfix This issue will be worked on labels Apr 29, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request establishes a continuous narrative across the Layer5 Cloud and Kanvas documentation, introducing a cast of fictional characters and organizations to provide context for tutorials and guides. The changes include a comprehensive design specification, a detailed implementation plan, and updates to existing content to integrate these narrative elements. Review feedback highlights the need for consistent capitalization of UI labels and terminology, proper punctuation in lists, the removal of hardcoded absolute paths in implementation scripts, and alignment of asset filenames between the design spec and the implementation plan.

1. Click the design you want to share (`microservices-baseline` in Five's case).
2. Click **Share**.
3. Enter Rex's email address (`rex@orbital-labs.example`).
4. Set the permission level — Five gives Rex **edit** access so he can deploy and modify the design.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The permission level "edit" should be capitalized as "Edit" to maintain consistency with other UI labels mentioned in the document (e.g., Share, Send, No access, Remove).

Suggested change
4. Set the permission level — Five gives Rex **edit** access so he can deploy and modify the design.
4. Set the permission level — Five gives Rex **Edit** access so he can deploy and modify the design.

Comment on lines +36 to +38
1. Open the design and click **Share**
2. Find Rex in the list of people with access
3. Change Rex's permission to **No access** or click **Remove**
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

These list items are missing trailing periods, which is inconsistent with the step-by-step instructions provided earlier in the document (lines 16-20).

Suggested change
1. Open the design and click **Share**
2. Find Rex in the list of people with access
3. Change Rex's permission to **No access** or click **Remove**
1. Open the design and click **Share**.
2. Find Rex in the list of people with access.
3. Change Rex's permission to **No access** or click **Remove**.

### External Links

Stumptown PBR&B keytar plaid street art, forage XOXO pitchfork selvage affogato green juice listicle pickled everyday carry hashtag. Organic sustainable letterpress sartorial scenester intelligentsia swag bushwick. Put a bird on it stumptown neutra locavore. IPhone typewriter messenger bag narwhal. Ennui cold-pressed seitan flannel keytar, single-origin coffee adaptogen occupy yuccie williamsburg chillwave shoreditch forage waistcoat.
- The design is shared with someone who doesn't have a Layer5 account or isn't signed in.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

Consider capitalizing "Account" to maintain consistency with existing terminology used in other parts of the documentation (e.g., "Layer5 Account").

Suggested change
- The design is shared with someone who doesn't have a Layer5 account or isn't signed in.
- The design is shared with someone who doesn't have a Layer5 Account or isn't signed in.

- [ ] **Step 1: Create the target directory**

```bash
mkdir -p /Users/l/code/docs/static/images/five
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The implementation plan contains hardcoded absolute paths (e.g., /Users/l/code/docs). This makes the plan non-portable and will cause failures for other contributors. Please use relative paths or environment variables.

Suggested change
mkdir -p /Users/l/code/docs/static/images/five
mkdir -p static/images/five

| `cloud/identity/users` | Five, Maya | Five's profile card; Maya as Org Admin | SVG 1 (meet Five) | Low | Open |
| `cloud/identity/organizations` | Maya, Marcus | Orbital Labs org creation; Stellar Dynamics as enterprise tenant | — | Low | Open |
| `cloud/identity/teams` | Zara, Rex, Jordan | Infrastructure vs. Dev team setup | team-of-fives.svg | Low | Open |
| `cloud/security/roles` | Dr. Aiko, Zara | Provider Admin → Org Admin → Team Admin chain | business-mode.svg | Medium | Open |
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The filename business-mode.svg is inconsistent with the filename layer5-five-mascot-means-business.svg used in the implementation plan. Please ensure the spec reflects the actual asset names.

Suggested change
| `cloud/security/roles` | Dr. Aiko, Zara | Provider Admin → Org Admin → Team Admin chain | business-mode.svg | Medium | Open |
| `cloud/security/roles` | Dr. Aiko, Zara | Provider Admin → Org Admin → Team Admin chain | `layer5-five-mascot-means-business.svg` | Medium | Open |

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

PR Preview Action v1.6.3
Preview removed because the pull request was closed.
2026-04-29 04:12 UTC

…976)

Signed-off-by: Lee Calcote <lee.calcote@layer5.io>
Signed-off-by: Lee Calcote <lee.calcote@layer5.io>
@leecalcote leecalcote force-pushed the feat/narrative-976-kanvas-gs branch from 49e0fb2 to 86ff367 Compare April 29, 2026 04:08
@leecalcote leecalcote changed the title [Docs] Update kanvas/getting-started to reference narrative characters [Docs] Update sharing-a-workspace, add promoting-a-design tutorial, and add kanvas/getting-started narrative callout Apr 29, 2026
@leecalcote leecalcote merged commit 346bd0a into master Apr 29, 2026
4 checks passed
@leecalcote leecalcote deleted the feat/narrative-976-kanvas-gs branch April 29, 2026 04:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/cloud-docs component/kanvas-docs help wanted Extra attention is needed issue/willfix This issue will be worked on

Projects

None yet

1 participant