Skip to content

feat(ui-pkgs): add deleteButtonProps to Show components #6699

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

Conversation

arndom
Copy link
Contributor

@arndom arndom commented Mar 7, 2025

PR Checklist

Please check if your PR fulfills the following requirements:

Bugs / Features

What is the current behavior?

Delete buttons in <Show/> components couldn't be customized via their props.

What is the new behavior?

Delete buttons in <Show/> components can now be customized via their props.

fixes (issue)
Resolves #6692

Notes for reviewers

I couldn't preview the docs locally. it failed during the process:
image

@arndom arndom requested a review from a team as a code owner March 7, 2025 09:29
Copy link

changeset-bot bot commented Mar 7, 2025

🦋 Changeset detected

Latest commit: 884e336

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@refinedev/chakra-ui Minor
@refinedev/ui-types Minor
@refinedev/mantine Minor
@refinedev/antd Minor
@refinedev/mui Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

netlify bot commented Mar 7, 2025

Deploy Preview for refine-doc-live-previews ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 57a3ff1
🔍 Latest deploy log https://app.netlify.com/sites/refine-doc-live-previews/deploys/67cabc6499068d00085c3e01
😎 Deploy Preview https://deploy-preview-6699--refine-doc-live-previews.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@BatuhanW
Copy link
Member

BatuhanW commented Mar 7, 2025

Hey @arndom thanks for the PR! We'll review it earliest next week. You can try pnpm run dev:docs command to start docs.

@arndom
Copy link
Contributor Author

arndom commented Mar 7, 2025

Thanks @BatuhanW. I still get the out of memory error, maybe my laptop can't run it

@alicanerdurmaz alicanerdurmaz added this to the March 2025 Release milestone Mar 10, 2025
@alicanerdurmaz alicanerdurmaz changed the base branch from main to next March 11, 2025 19:14
@alicanerdurmaz alicanerdurmaz requested a review from a team as a code owner March 12, 2025 02:15
@alicanerdurmaz
Copy link
Member

Hello @arndom, thanks for improving Refine!

The docs look good, but do you have time to write tests? For example, you changed the delete button's rendering condition, so we need to update this test. Also, maybe you could pass deleteButtonProps and test if it works.

@arndom
Copy link
Contributor Author

arndom commented Mar 12, 2025

my bad, skipped the tests, will make the updates 👍

@alicanerdurmaz alicanerdurmaz merged commit cd938b0 into refinedev:next Mar 24, 2025
13 checks passed
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.

[FEAT] Add deleteButtonProps in <Show>
3 participants