dify icon indicating copy to clipboard operation
dify copied to clipboard

chore: add redis for getting built plugin

Open LeeeeeeM opened this issue 6 months ago • 3 comments

Summary

It will cost long time getting all built plugins

Screenshots

in dev mode

Before After
image image

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.
  • [x] I've updated the documentation accordingly.
  • [x] I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

LeeeeeeM avatar May 22 '25 09:05 LeeeeeeM

@crazywoola CC

LeeeeeeM avatar May 22 '25 09:05 LeeeeeeM

Looks like miss the logic when the new BuiltinToolProvider been add into the db then need clear the redis cache. eg: in file: builtin_tools_manage_service.py

Dustyposa avatar May 23 '25 02:05 Dustyposa

Looks like miss the logic when the new BuiltinToolProvider been add into the db then need clear the redis cache. eg: in file: builtin_tools_manage_service.py

done

LeeeeeeM avatar May 26 '25 05:05 LeeeeeeM

Hi, thanks for the contribution. Why does it have to be closed? Any plan for reopening it and fixing the comments?

bowenliang123 avatar Jun 04 '25 07:06 bowenliang123