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.

Sign up for GitHub

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

Jump to bottom

fix: useForm warn condition #4926

Merged
merged 3 commits into from
Sep 11, 2023
Merged

fix: useForm warn condition #4926

merged 3 commits into from
Sep 11, 2023

Conversation

salihozdemir
Copy link
Contributor

Updated useForm warn condition. If the user passes the queryOption.enabled: false to the hook, the useForm will not throw a warning anymore.

@changeset-bot
Copy link

changeset-bot bot commented Sep 11, 2023 •
edited
Loading

🦋 Changeset detected

Latest commit: fe53539

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

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

@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

✅ Deploy Preview for lighthearted-pastelito-e86ad1 ready!

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/lighthearted-pastelito-e86ad1/deploys/64fee4a27f15190007a351e0
😎 Deploy Preview https://deploy-preview-4926--lighthearted-pastelito-e86ad1.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.

@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

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

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/refine-doc-live-previews/deploys/64fee4a25885340008042dfb
😎 Deploy Preview https://deploy-preview-4926--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.

@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

✅ Deploy Preview for imaginative-sundae-1fd394 ready!

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/imaginative-sundae-1fd394/deploys/64fee4a277482300083a8b11
😎 Deploy Preview https://deploy-preview-4926--imaginative-sundae-1fd394.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.

@nx-cloud
Copy link

nx-cloud bot commented Sep 11, 2023 •
edited
Loading

☁️ Nx Cloud Report

CI is running/has finished running commands for commit fe53539. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

🟥 Failed Commands
lerna run test --stream --scope @refinedev/* --scope create-refine-app
✅ Successfully ran 1 target

Sent with 💌 from NxCloud.

@refine-bot refine-bot temporarily deployed to deploy-preview-mui-4926 September 11, 2023 08:53 Inactive
@refine-bot refine-bot temporarily deployed to deploy-preview-antd-4926 September 11, 2023 08:54 Inactive
@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

✅ Deploy Preview for zippy-semolina-4ad81e ready!

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/zippy-semolina-4ad81e/deploys/64fee4a2218de700087d7e88
😎 Deploy Preview https://deploy-preview-4926--zippy-semolina-4ad81e.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.

@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

✅ Deploy Preview for stupendous-taffy-33f80c ready!

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/stupendous-taffy-33f80c/deploys/64fee4a2086657000715b550
😎 Deploy Preview https://deploy-preview-4926--stupendous-taffy-33f80c.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.

@netlify
Copy link

netlify bot commented Sep 11, 2023 •
edited
Loading

✅ Deploy Preview for aquamarine-panda-e208bf ready!

Name Link
🔨 Latest commit fe53539
🔍 Latest deploy log https://app.netlify.com/sites/aquamarine-panda-e208bf/deploys/64fee4a26da6ec0007082577
😎 Deploy Preview https://deploy-preview-4926--aquamarine-panda-e208bf.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.

@omeraplak omeraplak added this pull request to the merge queue Sep 11, 2023
Merged via the queue into next with commit 053798a Sep 11, 2023
44 of 46 checks passed
@omeraplak omeraplak deleted the fix/use-form-warn-condition branch September 11, 2023 15:59
This was referenced Sep 12, 2023
Merged
Merged
MahirMahdi pushed a commit to MahirMahdi/refine that referenced this pull request Sep 16, 2023
@salihozdemir @MahirMahdi
fix: useForm warn condition (refinedev#4926)
9519366 
* fix: update `useForm` warn condition

* chore: add changeset

* test: check warn messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@omeraplak omeraplak omeraplak approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@salihozdemir @omeraplak @refine-bot