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: 处理table.column渲染mapping值是 tagSchema 不正常渲染的情况, 避免渲染tag时从 props.va… #7058

Merged

Conversation

CheshireJCat
Copy link
Collaborator

@CheshireJCat CheshireJCat commented Jun 1, 2023

…lue 取值而无法渲染 label

What

🤖 Generated by Copilot at d6d2eb5

This pull request enhances the mapping field renderer to support tag schemas as values. It introduces a new custom renderer mapping-tag and adds a comment to explain the logic in Mapping.tsx.

🤖 Generated by Copilot at d6d2eb5

To render mapping fields with more grace
This pull request makes a small change
It adds a comment line
And uses mapping-tag fine
To show tag schemas in the right place

Why

How

🤖 Generated by Copilot at d6d2eb5

  • Add a comment to clarify the handling of tag schema values in mapping fields (link)
  • Change the rendering logic of mapping field labels to use a custom renderer for tag schemas and a default renderer for other schemas (link)

@github-actions
Copy link

github-actions bot commented Jun 1, 2023

👍 Thanks for this!
🏷 I have applied any labels matching special text in your issue.

Please review the labels and make any necessary changes.

@github-actions github-actions bot added the fix label Jun 1, 2023
@CheshireJCat CheshireJCat force-pushed the fix-table-column-mapping-render-tag-bug branch from d6d2eb5 to c75159b Compare June 1, 2023 08:05
@CheshireJCat CheshireJCat requested a review from 2betop June 1, 2023 08:10
@hsm-lv hsm-lv merged commit b477a6d into baidu:master Jun 1, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants