feat: document suggestions #2242
Conversation
There was a problem hiding this comment.
Please refer to the requested changes alongside the review for playground PRs.
One thing I am explicitly highlighting -
- Web: regression -> the feedback footer is shown even when streaming of AI answer is happening.
The expectations are
- Code change
- Loom explaining the code changes.
- Loom using the playrgound deployed previews(not locally) covering
- main effect
- please demo the AI related stories as well very thoroughly.
| }); | ||
|
|
||
|
|
||
| it('should render AI response & askButton & enterButton', async () => { |
There was a problem hiding this comment.
reactivesearch/packages/web/src/components/search/SearchBox.js
Lines 1061 to 1062 in acb9c23
We should change the class names for askbutton from enter-btn or enter-button-wrapper to ask-btn and ask-button-wrapper, please update the snapshot post this change
There was a problem hiding this comment.
What should I do here?
There was a problem hiding this comment.
@SavvyShah REfer to comment made for vue counterpart, this util function could be exported out from reactivecore
site/demos/airbeds/package.json
Outdated
| @@ -6,8 +6,8 @@ | |||
| "author": "metagrover", | |||
| "license": "Apache-2.0", | |||
| "dependencies": { | |||
| "@appbaseio/reactivemaps": "4.0.0", | |||
| "@appbaseio/reactivesearch": "4.0.5", | |||
| "@appbaseio/reactivemaps": "4.0.0-rc", | |||
There was a problem hiding this comment.
@SavvyShah any updates here?
Notion: Recent Document Suggestions
ReactiveCore PR
Docs PR
Adds snapshot tests for recent searchbox changes