Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(primeng/p-badge): rework dynamic property rerender for directive #12739

Merged

Conversation

volvachev
Copy link
Contributor

@volvachev volvachev commented Mar 11, 2023

Defect Fixes

Fixes #12736
Fixes #12959

@vercel
Copy link

vercel bot commented Mar 11, 2023

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

Name Status Preview Comments Updated (UTC)
primeng-ssr-test ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 18, 2024 7:26pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
primeng ⬜️ Ignored (Inspect) Visit Preview Apr 18, 2024 7:26pm

@volvachev volvachev force-pushed the feat-rework-badge-directive branch 2 times, most recently from 0356fca to 0abff48 Compare April 26, 2023 19:49
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test April 26, 2023 20:00 Inactive
@volvachev
Copy link
Contributor Author

Hi, @cetincakiroglu can you look?

@volvachev volvachev force-pushed the feat-rework-badge-directive branch from 0abff48 to 71bc3af Compare May 23, 2023 18:27
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test May 23, 2023 18:37 Inactive
@volvachev volvachev force-pushed the feat-rework-badge-directive branch from 71bc3af to a1311b0 Compare May 23, 2023 18:50
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test May 23, 2023 19:15 Inactive
@volvachev volvachev force-pushed the feat-rework-badge-directive branch from a1311b0 to 96d42ce Compare June 12, 2023 15:09
@vercel vercel bot temporarily deployed to Preview – primeng-ssr-test June 12, 2023 15:26 Inactive
@ProfEibe
Copy link

@cetincakiroglu anything we can do to get this merged?

@mertsincan mertsincan added Type: Breaking Change Issue contains a breaking change related to a specific component Status: Discussion Issue or pull request needs to be discussed by Core Team labels Aug 29, 2023
@mertsincan mertsincan added this to the 16.Future milestone Aug 29, 2023
@cetincakiroglu cetincakiroglu added the Status: Pending Review Issue or pull request is being reviewed by Core Team label Mar 22, 2024
@cetincakiroglu
Copy link
Contributor

Hi @volvachev,

Could you please resolve the conflicts and instead of removing the size property from badge which leads to breaking change, could you please deprecate it and add the new property?

I'll merge it after these changes.

@volvachev volvachev force-pushed the feat-rework-badge-directive branch from 96d42ce to 7b19c33 Compare April 18, 2024 19:26
@volvachev
Copy link
Contributor Author

Hi @volvachev,

Could you please resolve the conflicts and instead of removing the size property from badge which leads to breaking change, could you please deprecate it and add the new property?

I'll merge it after these changes.

Hi, I've updated

@cetincakiroglu cetincakiroglu merged commit 75c9cc0 into primefaces:master Apr 19, 2024
3 checks passed
@cetincakiroglu cetincakiroglu removed this from the 17.Future milestone Apr 19, 2024
@cetincakiroglu cetincakiroglu removed the request for review from gucal April 19, 2024 13:17
@cetincakiroglu cetincakiroglu removed Status: Pending Review Issue or pull request is being reviewed by Core Team Status: Discussion Issue or pull request needs to be discussed by Core Team Type: Breaking Change Issue contains a breaking change related to a specific component labels Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants