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

[type:fix] configs import and export refactor #5827

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eye-gu
Copy link
Contributor

@eye-gu eye-gu commented Dec 2, 2024

close #5826

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@eye-gu eye-gu force-pushed the fix-5826 branch 3 times, most recently from d4e8545 to 72fc338 Compare December 4, 2024 07:49
@eye-gu eye-gu requested a review from moremind December 4, 2024 08:20
/**
* dict.
*/
Dict("dict.json", 5),
Copy link
Member

Choose a reason for hiding this comment

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

may include dict and plugin handler?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently, it includes both of these

Copy link
Member

Choose a reason for hiding this comment

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

I think you should add plugin handler

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The plugin handler is associated with the plugin

/**
* dict.
*/
Dict("dict.json", 5),
Copy link
Member

Choose a reason for hiding this comment

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

I think you should add plugin handler

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.

[BUG] ID conflict during import config
2 participants