fix: sort import - #34
Merged
Merged
Conversation
rajat2502
approved these changes
Sep 7, 2025
rajat2502
left a comment
rajat2502
left a comment
Contributor
There was a problem hiding this comment.
looks good, please check why the tests are failing
Contributor
Author
Already fixed in parent PR. |
Tanya-atatakai
approved these changes
Sep 8, 2025
mohandast52
added a commit
that referenced
this pull request
Sep 9, 2025
* feat: add mock chat and types for agent responses * feat: add ui-chat library with chat components and configurations * feat: add GlobalColors and update time constants for better time management * feat: integrate chat feature with UnlockChat component and update feature hooks * feat: replace local NA constant with centralized import from util-constants-and-types * feat: refactor chat components and integrate mock chat functionality * feat: add chat assets and update GlobalColors; refactor ChatMarkdown component styles * feat: implement chat functionality with ViewChats component and update chat types * feat: refactor Chat component to use UiChat and remove unused imports; update index exports * feat: refactor Chat component to use UiChat and remove unused imports; update index exports * feat: implement Chat component with chat functionality and system message updates; add TradingStrategy and Pill components * feat: integrate Chat component into Agent; update Card styles and mockFeatures for chat functionality * feat: refactor chat components for improved styling and structure; update color constants and logo handling * feat: enhance Chat component with size prop and update ViewChats for responsive styling; add border-radius to CardV2 * feat: add Strategy component and integrate it into Agent; update Card component for dynamic padding * feat: update AgentDetailsCard to handle timestamp as string; enhance useAgentDetails with mockTraderInfo; refine mockAgentInfo and mockChat; adjust types for better flexibility * feat: integrate GLOBAL_COLORS into Pill and Chat components for consistent theming; utilize useMemo for button styles in Chat * feat: update mockFeatures to set isChatEnabled to false; add TODO comment in Pill component for future refactoring * fix: update test command in check-pull-request workflow to allow passing with no tests * fix: update trading_type references in Strategy and mockAgentInfo for consistency * fix: remove redundant return statement in getType function of TradingStrategy component * refactor: streamline chat message handling in Chat component; remove unnecessary console logs and error handling * refactor: rename btnStyles to commonBtnStyles for consistency in Chat component * refactor: reorganize imports and improve code structure across multiple files (#34) * refactor: remove mockChatList from mockChat.ts files to streamline code
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
Types of changes
What types of changes does your code introduce?
Put an
xin the boxes that apply