Skip to content

Conversation

@tataihono
Copy link
Contributor

@tataihono tataihono commented Jan 13, 2026

Add the 'Website' icon option to button leading/trailing icon selectors for admins.


Linear Issue: NES-1206

Open in Cursor Open in Web

Summary by CodeRabbit

  • New Features
    • Added a new "Website" icon option (LanguageRounded) available for selection across the editor and UI.
    • Backend API schemas updated to recognize the new icon name so it can be used throughout the application.

✏️ Tip: You can customize this high-level summary in your review settings.

- Add LanguageRounded to GraphQL IconName enum
- Update Icon component in journeys-ui to support LanguageRounded
- Add Website option to Icon selector in journeys-admin
- Update all generated GraphQL types

Resolves NES-1206
@cursor
Copy link

cursor bot commented Jan 13, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@linear
Copy link

linear bot commented Jan 13, 2026

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 13, 2026

Walkthrough

Added a new IconName enum member LanguageRounded, imported the Material-UI icon, and registered it in UI icon maps and admin icon selection so the new icon is available across schemas and components.

Changes

Cohort / File(s) Summary
GraphQL Schema Updates
apis/api-gateway/schema.graphql, apis/api-journeys-modern/schema.graphql, apis/api-journeys/schema.graphql, apis/api-journeys/src/app/modules/block/icon/icon.graphql
Added LanguageRounded to the IconName enum in four GraphQL schema files.
Admin UI icon list
apps/journeys-admin/src/components/Editor/Slider/Settings/CanvasDetails/Properties/controls/Icon/Icon.tsx
Imported LanguageRounded from @mui/icons-material/LanguageRounded and appended an icons array entry with value: IconName.LanguageRounded, label: t('Website'), and display: <LanguageRounded />.
Shared Icon component
libs/journeys/ui/src/components/Icon/Icon.tsx
Imported LanguageRounded and added it to the Icon component’s icon map so it can be rendered by iconName.
Minor import reorder
libs/journeys/ui/src/components/Step/Step.tsx
Reordered one import (useGetValueFromJourneyCustomizationString) without functional changes.

Sequence Diagram(s)

sequenceDiagram
  actor AdminUI as Admin UI (Editor)
  participant IconComp as Shared Icon Component
  participant GraphQL as GraphQL APIs
  participant DB as Database

  AdminUI->>IconComp: User selects IconName.LanguageRounded
  IconComp-->>AdminUI: render <LanguageRounded /> preview
  AdminUI->>GraphQL: Save block with iconName: LanguageRounded (mutation)
  GraphQL->>DB: persist iconName value
  DB-->>GraphQL: confirm saved
  GraphQL-->>AdminUI: mutation response (saved block)
  AdminUI->>IconComp: request render for saved block (iconName: LanguageRounded)
  IconComp-->>AdminUI: render <LanguageRounded />
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • jianwei1
  • mikeallisonJS
  • tanflem
🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: button website icon' directly summarizes the main change—adding a website icon option to button icon selectors across the codebase.
Linked Issues check ✅ Passed The pull request implements all coding requirements from NES-1206: backend GraphQL schema updates across three API files, icon component registration in two UI libraries, and admin UI integration for website icon selection.
Out of Scope Changes check ✅ Passed All changes are directly scoped to adding the LanguageRounded website icon support. The reordering of imports in Step.tsx is a minor organizational change that maintains the same functionality and does not introduce unrelated modifications.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cursor/NES-1206-button-website-icon-36fd

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 58951cc and 504d2ef.

⛔ Files ignored due to path filters (2)
  • apps/journeys-admin/__generated__/JourneyVisitorExportToGoogleSheet.ts is excluded by !**/__generated__/**
  • libs/shared/gql/src/__generated__/graphql-env.d.ts is excluded by !**/__generated__/**
📒 Files selected for processing (1)
  • libs/journeys/ui/src/components/Step/Step.tsx
✅ Files skipped from review due to trivial changes (1)
  • libs/journeys/ui/src/components/Step/Step.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (6)
  • GitHub Check: test (22, 2/3)
  • GitHub Check: Analyze (javascript)
  • GitHub Check: test (22, 1/3)
  • GitHub Check: test (22, 3/3)
  • GitHub Check: build (22)
  • GitHub Check: lint (22)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud
Copy link

nx-cloud bot commented Jan 13, 2026

View your CI Pipeline Execution ↗ for commit 504d2ef

Command Status Duration Result
nx run journeys-e2e:e2e ✅ Succeeded 45s View ↗
nx run videos-admin-e2e:e2e ✅ Succeeded 6s View ↗
nx run short-links-e2e:e2e ✅ Succeeded 5s View ↗
nx run journeys-admin-e2e:e2e ✅ Succeeded 2m 18s View ↗
nx run watch-e2e:e2e ✅ Succeeded 26s View ↗
nx run resources-e2e:e2e ✅ Succeeded 23s View ↗
nx run player-e2e:e2e ✅ Succeeded 3s View ↗
nx run-many --target=vercel-alias --projects=jo... ✅ Succeeded 2s View ↗
Additional runs (20) ✅ Succeeded ... View ↗

☁️ Nx Cloud last updated this comment at 2026-01-13 23:25:00 UTC

@cursor cursor bot marked this pull request as ready for review January 13, 2026 23:02
@tataihono tataihono changed the title Button website icon feat: button website icon Jan 13, 2026
@tataihono tataihono requested a review from jianwei1 January 13, 2026 23:03
@tataihono tataihono self-assigned this Jan 13, 2026
@github-actions github-actions bot temporarily deployed to Preview - short-links January 13, 2026 23:03 Inactive
@github-actions github-actions bot temporarily deployed to Preview - journeys January 13, 2026 23:03 Inactive
@github-actions github-actions bot temporarily deployed to Preview - videos-admin January 13, 2026 23:03 Inactive
@github-actions github-actions bot temporarily deployed to Preview - journeys-admin January 13, 2026 23:03 Inactive
@github-actions github-actions bot temporarily deployed to Preview - resources January 13, 2026 23:03 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
short-links ✅ Ready short-links preview Wed Jan 14 12:12:34 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
journeys ✅ Ready journeys preview Wed Jan 14 12:13:35 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
player ✅ Ready player preview Wed Jan 14 12:12:30 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
videos-admin ✅ Ready videos-admin preview Wed Jan 14 12:13:09 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
resources ✅ Ready resources preview Wed Jan 14 12:13:34 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
watch ✅ Ready watch preview Wed Jan 14 12:13:36 NZDT 2026

@github-actions
Copy link
Contributor

github-actions bot commented Jan 13, 2026

The latest updates on your projects.

Name Status Preview Updated (UTC)
journeys-admin ✅ Ready journeys-admin preview Wed Jan 14 12:14:26 NZDT 2026

@github-actions github-actions bot temporarily deployed to Preview - journeys January 13, 2026 23:10 Inactive
@github-actions github-actions bot temporarily deployed to Preview - videos-admin January 13, 2026 23:10 Inactive
@github-actions github-actions bot temporarily deployed to Preview - resources January 13, 2026 23:10 Inactive
@github-actions github-actions bot temporarily deployed to Preview - journeys-admin January 13, 2026 23:10 Inactive
@github-actions github-actions bot temporarily deployed to Preview - short-links January 13, 2026 23:10 Inactive
Copy link
Contributor

@jianwei1 jianwei1 left a comment

Choose a reason for hiding this comment

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

@cursoragent

  1. Add the new icon into libs/journeys/ui/src/components/Icon/Icon.stories.tsx , similar to what has been done in this PR: https://github.com/JesusFilm/core/pull/3008/files#diff-80a849c8f5205285cd4214e168416c9915d2d8aecc090be0689b10da2551ab2e

  2. Remove the changes that the lintbot made to libs/journeys/ui/src/components/Step/Step.tsx and apps/journeys-admin/generated/JourneyVisitorExportToGoogleSheet.ts, as these are unrelated to the changes

@cursor
Copy link

cursor bot commented Jan 14, 2026

❌ Unable to add follow-up after multiple attempts. The agent may be experiencing issues.

Please try again later or Open in Web to view the status.


Open in Cursor Open in Web

Learn more about Cursor Agents

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