Skip to content

558 update attach document button functionality#2701

Merged
neelneelneel merged 9 commits intodevfrom
558-update-attach-document-button-functionality
Feb 18, 2026
Merged

558 update attach document button functionality#2701
neelneelneel merged 9 commits intodevfrom
558-update-attach-document-button-functionality

Conversation

@ekim1707
Copy link
Copy Markdown
Contributor

@ekim1707 ekim1707 commented Feb 13, 2026

Description

updated behavior or document attachment between room and AskPlayground reactor

Changes Made

  • allowedFileTypes array comes in through themedb instead of being hardcoded in FE
  • all files are looped through, where their extensions are obtained, and they are compared to the file type array
  • file type array is normalized for matching

How to Test

  1. Set allowedFileTypes in themedb backend
  2. Upload attachment and ask Playground something ("describe the document")
  3. Look in UI File Explorer to see if all files attached
  4. Look in Network of browser to see if only filtered files were passed to AskPlayground

Notes

  • The functions are built to be forgiving to the extension names that come through. For example, this._theme.allowedFileTypes could be ["pdf", ".PDF", "", " png "], and this would eventually be reduced to { ".pdf", ".png" } for the final file extension checks. However, simplicity and consistency for the file type array should be stressed. A good example of a simple array in theme db would be [“pdf”, “png”, “jpg”].
  • Both jpg and jpeg must be explicitly stated in the array. A good example of which would be ["pdf", "png", "jpg", "jpeg"].

@ekim1707 ekim1707 requested review from ckelly17 and kzsb03 February 13, 2026 17:45
@ekim1707 ekim1707 self-assigned this Feb 13, 2026
@ekim1707 ekim1707 requested a review from a team as a code owner February 13, 2026 17:45
@neelneelneel neelneelneel merged commit fd2fc52 into dev Feb 18, 2026
3 checks passed
@neelneelneel neelneelneel deleted the 558-update-attach-document-button-functionality branch February 18, 2026 20:27
@github-actions
Copy link
Copy Markdown

@CodiumAI-Agent /update_changelog

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.

3 participants