radix-ui / icons

A crisp set of 15×15 icons designed by the @workos team.
https://radix-ui.com/icons
MIT License
2.25k stars 113 forks source link

add react 19 to peer deps #184

Closed jaredcbaum closed 1 month ago

jaredcbaum commented 3 months ago

This PR:

mxa0079 commented 3 months ago

Please merge

officiallerio commented 1 month ago

please merge

roelfjan commented 1 month ago

Merging this is one small step for a man, one giant leap for the community!

takefy-dev commented 1 month ago

Please merge it omg

codenomnom commented 1 month ago

Uhm, is there any issue merging this? 🤔

takefy-dev commented 1 month ago

bro is just AFKOn 24 Oct 2024, at 12:56, codenomnom @.***> wrote: Uhm, is there any issue merging this? 🤔

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

onurhan1337 commented 1 month ago

great work.

Gitstar-OC commented 1 month ago

Please merge it ?

needim commented 1 month ago

you can use overrides for a temporary fix.

"overrides": {
    "@radix-ui/react-icons": {
        "react": "$react"
    }
}

also overrides are needed for other libraries also, maybe define your overrides like this:

"overrides": {
    "react": "$react",
    "react-dom": "$react-dom"
}

[!WARNING]
This is just a temporary fix!

shadcn commented 1 month ago

friendly bump here. can we merge? any blockers?

rezarahem commented 1 month ago

This should have been merged already. No issues here.

takefy-dev commented 1 month ago

the owner is AFK he is not working at radix anymore

VibingCreator commented 1 month ago

Merge you tyrant.

guchris commented 1 month ago

is there someone we can contact?

officiallerio commented 1 month ago

is there someone we can contact?

the other stupid way we can do is by making nonsense PR on all radix repo by saying "merge the radix icons" for them to notice us

Bantolinao commented 1 month ago

please merge! why it take too long

DysektAI commented 1 month ago

Merge please, so the actual issues can be fixed...

Scholar01 commented 1 month ago

https://github.com/radix-ui/primitives/issues/3195 I went to another library and raised an issue

priyeshghamandi commented 1 month ago

Please merge

DariusLukasukas commented 1 month ago

Is this repo still maintained? Shadcn UI compatibility is blocked due to the pending Radix icons merge.

takefy-dev commented 1 month ago

No it’s not

On 26 Oct 2024, at 20:08, Darius Lukasukas @.***> wrote:

Is this repo still maintained? Shadcn UI compatibility is blocked due to the pending Radix icons merge.

— Reply to this email directly, view it on GitHub https://github.com/radix-ui/icons/pull/184#issuecomment-2439678947, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATANFWLBNTIKPV2MYR4PUOLZ5PLBNAVCNFSM6AAAAABNDCWC2OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGY3TQOJUG4. You are receiving this because you commented.

hsavit1 commented 1 month ago

cmon please merge this already

Saptak-Bose commented 1 month ago

merge this plsss...shadcn not working cause of this

officiallerio commented 1 month ago

try to put issues on other radix repo so that they will notice us.

hackerharsh-77 commented 1 month ago

pl merge this as it is causing version mismatch in frameworks react 19^

radioac7iv3 commented 1 month ago

Looks good to me, can we merge?

officiallerio commented 1 month ago

merge

Gitstar-OC commented 1 month ago

Please Merge @chaance and @hadihallak !!

shadowoff09 commented 1 month ago

Please merge it, shadcn is broken because of this.

officiallerio commented 1 month ago

@chaance @hadihallak please merge sir

priyeshghamandi commented 1 month ago

@chaance @hadihallak can you please merge this PR. It's blocking a lot of projects. Thanks,

tyrellshawn commented 1 month ago

@chaance @hadihallak Please merge its blocking

madebydor commented 1 month ago

merge it for the love of shadcn @chaance @hadihallak

officiallerio commented 1 month ago

:(

priyeshghamandi commented 1 month ago

@vladyslavmoroz @colmtuite @peduarte @loklaan can you please merge this PR. It's blocking a lot of projects. Thanks,

napstar-420 commented 1 month ago

Please merge it man...., i can't init the shadcn ui

diegohaz commented 1 month ago

I don't understand. Couldn't you just npm install --force while waiting for this?

peduarte commented 1 month ago

hide-hiding-gif