Skip to content

Drop fabricated testimonial names, fix Devstack Bridge crash, add changelog image - #9

Merged
harrymove-ctrl merged 1 commit into
masterfrom
fix/marketing-testimonials-and-devstack-bug
Jul 31, 2026
Merged

Drop fabricated testimonial names, fix Devstack Bridge crash, add changelog image#9
harrymove-ctrl merged 1 commit into
masterfrom
fix/marketing-testimonials-and-devstack-bug

Conversation

@harrymove-ctrl

Copy link
Copy Markdown
Owner

Summary

  • Testimonial wall on the landing page attributed real-sounding quotes to invented people ("Priya Sharma", "Aiko Nakamura", "Deniz Yilmaz"). Replaced each with a generic role-only attribution since none of these are real, verifiable people.
  • DevstackService.getDeployment() assumed published packages in deployment.json were plain id strings. Newer devstack CLI versions (0.4.0+) write { "counter": { "id": "0x..." } } instead, which crashed the Devstack Bridge page's React render (Objects are not valid as a React child). Now accepts both shapes.
  • Added the Devstack Bridge screenshot to the v1.6.0 changelog entry's improvementsImage slot.

Test plan

  • npm run build -w sui-cli-web-server
  • npm run build -w @sui-cli-web/marketing
  • Verified the Devstack Bridge page renders real deployment data (RPC, package id, funded account) against a live local devstack project instead of crashing
  • Verified the changelog page renders the new image in the right spot

…, add Devstack Bridge changelog image

- Testimonial wall: "Priya Sharma", "Aiko Nakamura" and "Deniz Yilmaz" were
  invented people with invented quotes attributed to them. Replaced the
  attribution with their generic role only.
- DevstackService.getDeployment(): newer devstack CLI versions emit
  published packages as an object with an id field instead of a bare id
  string, which crashed the Devstack Bridge page's render. Accept both shapes.
- Changelog: add the Devstack Bridge screenshot to the v1.6.0 entry's
  improvements image slot.
@harrymove-ctrl
harrymove-ctrl merged commit ce85be5 into master Jul 31, 2026
2 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.

2 participants