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 Paste into chat box retains all formatting #119

Merged
merged 4 commits into from
Feb 27, 2025

Conversation

nboyse
Copy link
Collaborator

@nboyse nboyse commented Feb 26, 2025

Context

As a user
When I paste copied information into the redbox chat box
I want all formatting to be retained
So that I do not have to manually re-add formatting

Acceptance:
When text contains A1, A2, and Bullet Point content and is copied from its originating document and pasted into redbox input box, Redbox should respect that formatting when the user presses enter.

Changes proposed in this pull request

Switching to a div in which supports bullet points instead of a textarea and updating the relevant js/css to ensure functionality and appearance still works as expected.

Guidance to review

Does it work for you?

Relevant links

https://uktrade.atlassian.net/browse/REDBOX-409

Things to check

  • I have added any new ENV vars in all deployed environments
  • I have tested any code added or changed
  • I have run integration tests

Copy link
Contributor

@larry6point6 larry6point6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally all good

@nboyse nboyse merged commit e598a5f into dev Feb 27, 2025
7 checks passed
@nboyse nboyse deleted the feature/allow-formatting-to-be-retained branch February 27, 2025 14:39
nboyse added a commit that referenced this pull request Feb 28, 2025
* Spike LLM to choose route (#113)

* spike LLM to choose route

* add explanation why choosing metadata prompt

---------

Co-authored-by: Saisakul Chernbumroong <[email protected]>

* turn self route using env (#121)

* turn self route using env

* add SELF_ROUTE_ENABLED to env.example

---------

Co-authored-by: Saisakul Chernbumroong <[email protected]>

* Bugfix/search irrelevant query (#118)

* remove bm25 query

* understanding queries

* move notebook

* moving notebook to notebooks folder

---------

Co-authored-by: nora-errouhly <[email protected]>

* Add summarise graph (#115)

* add summarise graph

* add new summarise graph

* add keyword to test

---------

Co-authored-by: Saisakul Chernbumroong <[email protected]>

* feat: Copy Paste into chat box retains all formatting (#119)

* Feature/new search graph (#120)

* add new search graph

* add new search graph

* move set route node. Add clear documents.

* remove temp search

---------

Co-authored-by: Saisakul Chernbumroong <[email protected]>

* Feature/similarity score threshold (#127)

* remove bm25 query

* understanding queries

* move notebook

* moving notebook to notebooks folder

* setting similar score threshold

* linting

* fixing unit tests

---------

Co-authored-by: nora-errouhly <[email protected]>
Co-authored-by: Tash Boyse <[email protected]>
Co-authored-by: Natasha Boyse <[email protected]>

* feat: Intercept incompatible (not UTF-8) text on upload and handle .doc (#123)

* feat: Intercept incompatible (not UTF-8) text on upload and add functionality to convert doc to docx

* switch to if instead of elif in case both scenarios apply

---------

Co-authored-by: Saisakul Chernbumroong <[email protected]>
Co-authored-by: Saisakul Chernbumroong <[email protected]>
Co-authored-by: Nora Er-Rouhly <[email protected]>
Co-authored-by: nora-errouhly <[email protected]>
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.

2 participants