Skip to content

fix(hub): add missing dalas icon#2838

Closed
jog1t wants to merge 1 commit into
mainfrom
08-04-fix_hub_add_missing_dalas_icon
Closed

fix(hub): add missing dalas icon#2838
jog1t wants to merge 1 commit into
mainfrom
08-04-fix_hub_add_missing_dalas_icon

Conversation

@jog1t
Copy link
Copy Markdown
Contributor

@jog1t jog1t commented Aug 4, 2025

Closes FRONT-710

Changes

  • Added Dallas, Texas, USA (tr-d) as a new region with the US flag emoji 🇺🇸
  • Improved TypeScript type safety for region-related functions and objects:
    • Added satisfies Record<string, IconProp | string> to REGION_ICON
    • Added satisfies Record<keyof typeof REGION_ICON, string> to REGION_LABEL
    • Updated getRegionKey return type to keyof typeof REGION_ICON
    • Fixed type casting in RegionIcon component

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Aug 4, 2025

Deploying rivet-studio with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9bf4346
Status:🚫  Build failed.

View logs

@vercel
Copy link
Copy Markdown

vercel Bot commented Aug 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rivet-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 4, 2025 9:04pm

Copy link
Copy Markdown
Contributor Author

jog1t commented Aug 4, 2025


How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Aug 4, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@jog1t jog1t marked this pull request as ready for review August 4, 2025 20:55
@linear
Copy link
Copy Markdown

linear Bot commented Aug 4, 2025

@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app Bot commented Aug 4, 2025

Merge activity

  • Aug 4, 8:56 PM UTC: jog1t added this pull request to the Graphite merge queue.
  • Aug 4, 8:56 PM UTC: CI is running for this pull request on a draft pull request (#2839) due to your merge queue CI optimization settings.
  • Aug 4, 8:57 PM UTC: Merged by the Graphite merge queue via draft PR: #2839.

graphite-app Bot pushed a commit that referenced this pull request Aug 4, 2025
Closes FRONT-710

## Changes

- Added Dallas, Texas, USA (`tr-d`) as a new region with the US flag emoji 🇺🇸
- Improved TypeScript type safety for region-related functions and objects:
  - Added `satisfies Record<string, IconProp | string>` to `REGION_ICON`
  - Added `satisfies Record<keyof typeof REGION_ICON, string>` to `REGION_LABEL`
  - Updated `getRegionKey` return type to `keyof typeof REGION_ICON`
  - Fixed type casting in `RegionIcon` component
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying rivet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9bf4346
Status:🚫  Build failed.

View logs

@graphite-app graphite-app Bot closed this Aug 4, 2025
@graphite-app graphite-app Bot deleted the 08-04-fix_hub_add_missing_dalas_icon branch August 4, 2025 20:57
@graphite-app
Copy link
Copy Markdown
Contributor

graphite-app Bot commented Aug 4, 2025

Graphite Automations

"Test" took an action on this PR • (08/04/25)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

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.

1 participant