Skip to content

fix: don't call defaultValues in form objects twice#46

Merged
Akryum merged 1 commit intomainfrom
fix/default-values-duplicate-call
Jul 22, 2025
Merged

fix: don't call defaultValues in form objects twice#46
Akryum merged 1 commit intomainfrom
fix/default-values-duplicate-call

Conversation

@Akryum
Copy link
Copy Markdown
Collaborator

@Akryum Akryum commented Jul 22, 2025

Description

Additional context


What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Before submitting the PR, please make sure you do the following

  • If it's a new feature, provide a convincing reason to add it. Ideally, you should open a suggestion issue first and have it approved before working on it.
  • Read the Contributing Guidelines.
  • Read the Pull Request Guidelines and follow the Commit Convention.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary of Changes

Hello @Akryum, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request resolves a bug in the form object creation process within the Vue package. It specifically targets and corrects an inefficiency where the defaultValues function was being invoked more than once, ensuring that form initialization is now more efficient and accurate by directly utilizing the pre-processed initialData.

Highlights

  • Bug Fix: Addressed an issue where the defaultValues function was being called redundantly during the initialization of form objects. The fix ensures that initialData is used directly, preventing multiple invocations of defaultValues.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments or fill out our survey to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Jul 22, 2025

Open in StackBlitz

@rstore/core

npm i https://pkg.pr.new/directus/rstore/@rstore/core@46

@rstore/nuxt

npm i https://pkg.pr.new/directus/rstore/@rstore/nuxt@46

@rstore/nuxt-directus

npm i https://pkg.pr.new/directus/rstore/@rstore/nuxt-directus@46

@rstore/nuxt-drizzle

npm i https://pkg.pr.new/directus/rstore/@rstore/nuxt-drizzle@46

@rstore/shared

npm i https://pkg.pr.new/directus/rstore/@rstore/shared@46

@rstore/vue

npm i https://pkg.pr.new/directus/rstore/@rstore/vue@46

commit: 1874491

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request correctly fixes an issue where defaultValues was called twice during form object creation by caching the result in a variable. My review includes one suggestion to improve the type safety of the change by removing a slightly inaccurate type assertion.

@Akryum Akryum merged commit e3d234d into main Jul 22, 2025
8 checks passed
@Akryum Akryum deleted the fix/default-values-duplicate-call branch October 25, 2025 15:23
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.

1 participant