-
Notifications
You must be signed in to change notification settings - Fork 35
Remove extraneous verify information message & tweak initial message #1585
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for docs-frontend-stg ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for atlas ready
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR removes the mongoDbVerifyInformationMessage from the chatbot modal and replaces it with an improved initial welcome message. The changes streamline the user experience by removing redundant bottom text and providing clearer, more comprehensive guidance upfront.
- Removed
inputBottomTextprop and its associatedmongoDbVerifyInformationMessageimport from the ModalView - Updated
initialMessageTextto provide a more detailed and informative welcome message that explains the assistant's purpose - Cleaned up the test mock to remove the now-unused
mongoDbVerifyInformationMessageexport
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/components/ActionBar/ChatbotModal.tsx | Removed mongoDbVerifyInformationMessage import and inputBottomText prop; enhanced the initial welcome message with clearer instructions |
| tests/unit/ActionBar.test.js | Removed mock for mongoDbVerifyInformationMessage to align with the source code changes |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
biancalaube
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Visually looks great, however there seems to be some linting errors. Could you run npm run lint && npm run format. Also, our release cycle is on Wednesday so once this is approved you won't see the changes in prod until next Wednesday just to let you know.
ASteinheiser
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree with @biancalaube , looks good, just need to run pnpm lint:fix
|
@biancalaube @ASteinheiser Thanks! Unfortunately I'm not able to |
Stories/Links:
Current Behavior:
Put a link to current staging or production behavior, if applicable
Staging Links:
Put a link to your staging environment(s), if applicable
Notes:
README updates