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

feat: Copy workspace info #8265

Closed
wants to merge 1 commit into from

Conversation

cuckootan
Copy link
Contributor

Checklist:

Important

Please review the checklist below before submitting your pull request.

  • Please open an issue before creating a PR or link to an existing issue
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

Description

add copy button to copy workspace basic information(name and id)

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update, included: Dify Document
  • Improvement, including but not limited to code refactoring, performance optimization, and UI/UX improvement
  • Dependency upgrade

Testing Instructions

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • copy workspace id and name into clipboard
image image

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. ☕️ typescript Pull request that update TypeScript code. 💪 enhancement New feature or request labels Sep 11, 2024
@cuckootan cuckootan changed the title copy workspace info feat: Copy workspace info Sep 11, 2024
@crazywoola
Copy link
Member

I am not sure that why do you need this button?

@cuckootan cuckootan force-pushed the dev/copy_workspace_info branch from 689e902 to c3404ac Compare September 11, 2024 12:08
@cuckootan
Copy link
Contributor Author

I am not sure that why do you need this button?

We deploy dify as a llm platform in our corp, and is widely used by many colleagues. So, it is necessary to create multiple workspaces to manage their usages according to difference business teams. In addition, we deploy an online model service that its codes are extracted from dify. It is used to manage model api call to all model providers configured in a certain workspace on dify. That means users in the workspace have to know workspace info(id, etc.) and specify model name they want to use. Then our service fetch the model credentials according to workspace id and model name, and finally call the model api.

@crazywoola
Copy link
Member

Ok, I need to forward this to our UI/UX designer to see if this is acceptable.

@crazywoola
Copy link
Member

Sorry our designer does not agree with this feature.

@crazywoola crazywoola closed this Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💪 enhancement New feature or request size:S This PR changes 10-29 lines, ignoring generated files. ☕️ typescript Pull request that update TypeScript code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants