dify icon indicating copy to clipboard operation
dify copied to clipboard

Remove postgresql-specific default values in model definition

Open whhe opened this issue 10 months ago • 4 comments

Summary

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

[!Tip] Close issue syntax: Fixes #<issue number> or Resolves #<issue number>, see documentation for more details.

Resolves #15411

Screenshots

Before After
... ...

Checklist

[!IMPORTANT]
Please review the checklist below before submitting your pull request.

  • [ ] This change requires a documentation update, included: Dify Document
  • [x] I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • [x] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • [ ] I've updated the documentation accordingly.
  • [x] I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

whhe avatar Mar 12 '25 11:03 whhe

I think this will introduce breaking changes to our existing system, please provide a fallback plan as well.

crazywoola avatar Mar 12 '25 11:03 crazywoola

I think this will introduce breaking changes to our existing system, please provide a fallback plan as well.

This change is not expected to introduce changes to the table structure. Which part do you think may break the existing design?

whhe avatar Mar 12 '25 12:03 whhe

@laipz8200 Thoughts?

crazywoola avatar Mar 13 '25 01:03 crazywoola

will check it later

laipz8200 avatar Mar 13 '25 05:03 laipz8200

Moved to https://github.com/langgenius/dify/pull/16264.

whhe avatar Mar 19 '25 20:03 whhe