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

fix: add default parameter configuration #2670

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

anjiazhuyouxing
Copy link
Contributor

@anjiazhuyouxing anjiazhuyouxing commented Jan 13, 2025

中文模板 / Chinese Template

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Test Case
  • TypeScript definition update
  • Document improve
  • CI/CD improve
  • Branch sync
  • Other, please describe:

PR description

修复未配置默认参数问题

Changelog

🇨🇳 Chinese

  • Fix: 修复JsonViewer未配置默认参数问题
  • Test: 更新JsonViewer测试用例

🇺🇸 English

  • Fix: Fix the problem that JsonViewer is not configured with default parameters
  • Test: update e2e test cases

Checklist

  • Test or no need
  • Document or no need
  • Changelog or no need

Other

  • Skip Changelog

Additional information

Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8f6c1b8:

Sandbox Source
pr-story Configuration

@YannLynn YannLynn changed the base branch from release to main January 13, 2025 08:51
@YannLynn YannLynn self-requested a review January 13, 2025 08:55
@YannLynn YannLynn merged commit 9e9bf77 into DouyinFE:main Jan 13, 2025
6 of 8 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.

2 participants