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(chat): fix API request when manipulating side panel folders (Issue #3133) #3285

Open
wants to merge 53 commits into
base: development
Choose a base branch
from

Conversation

Derikyan
Copy link
Contributor

@Derikyan Derikyan commented Mar 3, 2025

Description:

Actual result: Get request is sent /api/listing/conversations/${bucket}/New%20folder%202?filter=ITEM&recursive=true
Expected result: no API request should be sent when manipulate with side panel folder

Issues:

Checklist:

  • the pull request name complies with Conventional Commits
  • the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
  • the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
  • I confirm that do not share any confidential information like API keys or any other secrets and private URLs

IlyaBondar and others added 30 commits December 10, 2024 18:15
@Derikyan Derikyan added the bug Something isn't working label Mar 3, 2025
@Derikyan Derikyan self-assigned this Mar 3, 2025
@Derikyan
Copy link
Contributor Author

Derikyan commented Mar 3, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3285.nightly-test.deltixhub.io
E2E tests status: failed

@Derikyan
Copy link
Contributor Author

Derikyan commented Mar 4, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3285.nightly-test.deltixhub.io
E2E tests status: failed

@irinakartun
Copy link
Contributor

irinakartun commented Mar 4, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3285.nightly-test.deltixhub.io
E2E tests status: failed

@Derikyan
Copy link
Contributor Author

Derikyan commented Mar 4, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3285.nightly-test.deltixhub.io
E2E tests status: failed

@irinakartun
Copy link
Contributor

irinakartun commented Mar 4, 2025

/deploy-review

Environment URL: https://chat-ai-dial-chat-pr-3285.nightly-test.deltixhub.io
E2E tests status: failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants