✨ feat: add xpay icon#262
Conversation
Reviewer's GuideAdds the new Xpay provider icon package (mono, color, text, combined, avatar variants) and wires it into the icon system, provider mapping, enum, exports, docs, and table of contents. Class diagram for Xpay icon componentsclassDiagram
direction LR
class IconType {
<<interface>>
+render(props)
}
class XpayMono {
+IconType(props)
}
class XpayColor {
+IconType(props)
}
class XpayText {
+IconType(props)
}
class IconCombineProps {
+number size
+number spaceMultiple
+number textMultiple
+string ariaLabel
}
class CombineProps {
+string type
}
class IconCombine {
+render(Icon, Text, size, spaceMultiple, textMultiple, ariaLabel)
}
class XpayCombine {
+CombineProps props
+render(type, size, spaceMultiple, textMultiple, ariaLabel)
}
class IconAvatarProps {
+number size
+string background
+string color
+number iconMultiple
+string ariaLabel
}
class AvatarProps {
}
class IconAvatar {
+render(Icon, size, background, color, iconMultiple, ariaLabel)
}
class XpayAvatar {
+AvatarProps props
+render(size, background, color, iconMultiple, ariaLabel)
}
class XpayStyles {
+string TITLE
+number COMBINE_TEXT_MULTIPLE
+number COMBINE_SPACE_MULTIPLE
+string COLOR_PRIMARY
+string AVATAR_BACKGROUND
+string AVATAR_COLOR
+number AVATAR_ICON_MULTIPLE
}
class XpayCompoundedIcon {
+XpayColor Color
+XpayText Text
+XpayCombine Combine
+XpayAvatar Avatar
+string colorPrimary
+string title
}
class XpayDefaultExport {
+XpayColor Color
+XpayText Text
+XpayCombine Combine
+XpayAvatar Avatar
+string colorPrimary
+string title
}
XpayMono ..|> IconType
XpayColor ..|> IconType
XpayText ..|> IconType
XpayCombine ..> IconCombineProps
XpayCombine ..> IconCombine
XpayCombine ..> XpayColor
XpayCombine ..> XpayMono
XpayCombine ..> XpayText
XpayAvatar ..> IconAvatarProps
XpayAvatar ..> IconAvatar
XpayAvatar ..> XpayMono
XpayStyles <.. XpayCombine
XpayStyles <.. XpayAvatar
XpayStyles <.. XpayDefaultExport
XpayCompoundedIcon <|.. XpayDefaultExport
XpayMono <|-- XpayDefaultExport
Class diagram for Xpay provider wiring into icon systemclassDiagram
direction LR
class ModelProvider {
<<enum>>
Xpay
Xinference
Fal
Ai302
AiHubMix
}
class XpayDefaultExport
class XpayIconExport {
+Xpay: XpayDefaultExport
}
class ProviderMapping {
+Icon: any
+combineMultiple: number
+keywords: ModelProvider[]
}
class ProviderMappingsConfig {
+providerMappings: ProviderMapping[]
}
class IconTocParam {
+hasAvatar: bool
+hasBrand: bool
+hasBrandColor: bool
+hasColor: bool
+hasCombine: bool
+hasText: bool
+hasTextCn: bool
+hasTextColor: bool
}
class IconToc {
+id: string
+title: string
+fullTitle: string
+group: string
+color: string
+desc: string
+docsUrl: string
+param: IconTocParam
}
class TocConfig {
+toc: IconToc[]
}
class XpayTocEntry {
+id_Xpay: string
+group_provider: string
+color_0F1C4D: string
+desc_https_xpay_sh: string
+docsUrl_xpay: string
}
class XpayDocPage {
+icons: example
+text: example
+combine: example
+avatars: example
+colors: example
}
XpayIconExport --> XpayDefaultExport
ProviderMappingsConfig "1" o--> "*" ProviderMapping
ProviderMapping --> ModelProvider
ProviderMapping --> XpayDefaultExport : Icon
TocConfig "1" o--> "*" IconToc
IconToc --> IconTocParam
XpayTocEntry <|-- IconToc
XpayDocPage --> XpayDefaultExport
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
👍 @sriakula1 |
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- In
src/toc.ts, consider settinghasBrandColor: truefor Xpay since you exposecolorPrimaryand add a Colors section in the docs, so the UI can surface the brand color consistently with other providers. - You already define
COLOR_PRIMARY(and related avatar colors) insrc/Xpay/style.ts, butcomponents/Color.tsxre-embeds the hex values; importing and using the shared constants there would avoid duplication and make future brand color updates safer.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In `src/toc.ts`, consider setting `hasBrandColor: true` for Xpay since you expose `colorPrimary` and add a Colors section in the docs, so the UI can surface the brand color consistently with other providers.
- You already define `COLOR_PRIMARY` (and related avatar colors) in `src/Xpay/style.ts`, but `components/Color.tsx` re-embeds the hex values; importing and using the shared constants there would avoid duplication and make future brand color updates safer.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Hi, could you review the request if it could be merged please? |
* ✨ feat: Add LLM API logo * ✨ feat: add xpay icon * fix: set hasBrandColor to true in toc entry --------- Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
|
ref: #276 |
* ✨ feat(icons): add Qoder brand icon (#270) * ✨ feat(icons): add Qoder brand icon - Add Qoder icon components (Avatar, Color, Mono, Text) - Add Qoder icon documentation - Export Qoder icon in icons.ts - Add Qoder entry in table of contents new file: src/Qoder/components/Avatar.tsx new file: src/Qoder/components/Color.tsx new file: src/Qoder/components/Mono.tsx new file: src/Qoder/components/Text.tsx new file: src/Qoder/index.md new file: src/Qoder/index.ts new file: src/Qoder/style.ts modified: src/icons.ts modified: src/toc.ts * ✨ feat: add Qoder icon - Add Qoder icon with Avatar, Color, Mono, and Text variants - Export Qoder icon from react-native package * ✨ feat: add Coding Plan provider icons support (#272) * ✨ feat: add Coding Plan provider support * ✨ feat: add Coding Plan provider support for React Native * feat: add Apertis icon (#269) * feat: add Apertis icon Add Apertis (apertis.ai) — a unified API platform for 500+ AI models across 30+ providers. Includes Mono, Text, Avatar, and Combine components following the standard icon package structure. Primary color: #0d9488 (Apertis Teal) * feat: export Apertis icon and add react-native support Add Apertis to src/icons.ts barrel export and create the full react-native icon set (Mono, Text, Avatar, Combine) under packages/react-native/src/icons/Apertis. * Update Together AI logo (#268) * wip * wip * Revert "wip" This reverts commit 76dfa33. * wip * feat: add AskVerdict AI brand icons (#265) Add AskVerdict AI (askverdict.ai) — multi-agent AI debate engine. Components: Mono, Color (gold gradient), Text, Combine, Avatar Shared SVG paths extracted to paths.ts to prevent divergence. Brand color: #E8A317 (gold) Website: https://askverdict.ai * ✨ feat: add AgentVoice icon (#257) Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * feat: merge PR #259 (LLM API) and PR #262 (Xpay) into dev (#276) * ✨ feat: Add LLM API logo * ✨ feat: add xpay icon * fix: set hasBrandColor to true in toc entry --------- Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com> Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> * 💄 style: Fix some icon style * ✨ feat: Update rn --------- Co-authored-by: zhi <yantze@126.com> Co-authored-by: Hardy <yimingmail@yandex.com> Co-authored-by: Quert <a97041304@gmail.com> Co-authored-by: Hassan El Mghari <hassan4709@gmail.com> Co-authored-by: GDS K S <39922405+thegdsks@users.noreply.github.com> Co-authored-by: Ryan Howard <ryan@agentvoice.com> Co-authored-by: RRyanHoward <ryan@websitellm.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: Markos Basilio <markosbasilio@MacBook-Pro-Markos.local> Co-authored-by: xPay <xpaysh@gmail.com>
💻 变更类型 | Change Type
🔀 变更说明 | Description of Change
Add xpay provider icon with all standard variants:
{x✦}icon)type='color'|'mono')Brand colors:
#0F1C4D(navy primary),#00DC9C(turquoise),#FFB400(gold). Registered asprovidergroup.Also registers
Xpayin the provider config, provider enum, icon exports, and table of contents.📝 补充信息 | Additional Information
Files added:
src/Xpay/components/Avatar.tsxsrc/Xpay/components/Color.tsxsrc/Xpay/components/Combine.tsxsrc/Xpay/components/Mono.tsxsrc/Xpay/components/Text.tsxsrc/Xpay/index.tssrc/Xpay/index.mdsrc/Xpay/style.tsFiles modified:
src/features/providerConfig.tsx— Added Xpay to provider configsrc/features/providerEnum.ts— Added Xpay enum entrysrc/icons.ts— Added Xpay exportsrc/toc.ts— Added Xpay to table of contentsBrand reference: brandfetch.com/xpay.sh
Summary by Sourcery
Add Xpay as a new provider icon with full variants and integrate it into the icon library and provider mappings.
New Features:
Documentation: