generated from amazon-archives/__template_Apache-2.0
-
Notifications
You must be signed in to change notification settings - Fork 49
Modified Files Tracker component #444
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
Open
sacrodge
wants to merge
84
commits into
aws:feature/upgrade-modified-files-ux
Choose a base branch
from
sacrodge:feature-prod
base: feature/upgrade-modified-files-ux
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+897
−4
Open
Changes from 72 commits
Commits
Show all changes
84 commits
Select commit
Hold shift + click to select a range
84dc8f4
created new component to track modified files along with tests
sacrodge 9cf60fe
refactored mynah-ui code as it had some duplication of code
sacrodge 10e5b09
fixed formatting issues
sacrodge 0f81966
working version for demo. Still no functionalities added like onClick…
sacrodge ca53a64
working version for demo. Still no functionalities added like onClick…
sacrodge 3fa96d1
feat: add onClick and undo functionalities
sacrodge cdc71c2
feat: add onClick and undo functionalities
sacrodge eb725c9
feat: add onClick and undo functionalities
sacrodge d2096a1
msg: working-version : Added onFileClick functionality to open the fi…
sacrodge b752e8d
msg: working-version : Added onFileClick functionality to open the fi…
sacrodge 50890e4
msg: working-version : Added onFileClick functionality to open the fi…
sacrodge 10adc33
msg: working-version : Added onFileClick functionality to open the fi…
sacrodge 38c0200
msg: working-version : Added onFileClick functionality to open the fi…
sacrodge 68cb763
msg: refactoring code to use existing functionality
sacrodge 5650932
msg: refactoring code to use existing functionality
sacrodge 3ad8e8d
msg: arrow aligns with titleText
sacrodge 0e8a803
msg: not working, imported file functionality from chat-item-card
sacrodge 0144697
msg: not working, imported file functionality from chat-item-card
sacrodge d2659ad
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge fe41190
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge 63695b8
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge 1f1137f
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge 37a3337
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge 298d4f5
msg: working-mynahUI, Lists files with undo buttons. refactored code
sacrodge e667f83
msg: working, arrow now alogns with titleText
sacrodge 516842e
msg: working, with no edits to language-servers
sacrodge 8ca1a61
msg: working, with no edits to language-servers
sacrodge ca1c746
msg: UI works with files displayed as pills. not-tested with LS.
sacrodge a39316c
msg: UI works with files displayed as pills. not-tested with LS.
sacrodge 2cf6ce6
msg: not-working, styled pills for better looks
sacrodge ff02f3a
msg: working
sacrodge d8b515f
msg: working
sacrodge 0cf483d
msg: working-fully, but missing functionalities
sacrodge 5827791
msg: working-fully, but missing functionalities
sacrodge 1c69d1e
msg: working-fully, but missing functionalities
sacrodge 2410e64
msg: working model;
sacrodge a31f1c8
msg: working model;
sacrodge 865e71e
msg: working model;
sacrodge 9945b21
msg: working;
sacrodge dc36e82
msg: working;
sacrodge c144998
msg: Working;
sacrodge 59930b5
msg: Working;
sacrodge aef2e28
msg: working;
sacrodge 608c329
msg: working;
sacrodge 82b5f57
msg: working;
sacrodge 4d9087a
msg: partially-working;
sacrodge 91627ab
msg: partially-working;
sacrodge c6c18e4
msg: working:
sacrodge 1ee0c80
msg: working;
sacrodge 7ed1a55
build: working;
sacrodge a1eb653
msg: working;Added
sacrodge bae7b41
msg: partially-working;
sacrodge b059e5b
msg: working:completely
sacrodge ae378ad
msg: WORKING;COMPLETELY;
sacrodge d1622b6
msg:working-partially;
sacrodge b27b664
msg:working-partially;
sacrodge 9e48653
msg: working-partially
sacrodge 0a9c0bd
msg:working-partially
sacrodge 59d40c6
msg:working-partially
sacrodge ea33df3
msg:working-completely;
sacrodge 1fd9b6d
msg:working; removed unnecessary styling
sacrodge ae513bd
msg:working; removed unnecessary styling
sacrodge 5e50abe
msg: Working;
sacrodge e1a040a
msg:working but not tested
sacrodge 55a1081
msg:working-partially; Reverted unnecessary data structures
sacrodge a6e1b38
msg: working-partially with LS;
sacrodge b4c9240
msg: working-partially;
sacrodge 635e0d8
msg: working-partially;
sacrodge 3e11fc5
msg: working-partially;
sacrodge 8d4e9d5
msg: working-partially;
sacrodge 0c482da
msg: working-partially;
sacrodge 023480d
msg: working-partially;
sacrodge cd11065
msg: working-partially;
sacrodge cd411f2
msg: working-completely;
sacrodge dc3f6a2
msg: working-completely;
sacrodge 5535430
msg: working-partially;
sacrodge 5aa50cd
msg: added comprehensive tests
sacrodge ce1482c
msg: E2E tests were failing because of the new component changing the…
sacrodge 5b8b76a
msg: working-completely;
sacrodge 62451f9
msg: working-completely;
sacrodge 25a27b6
msg: working-completely:
sacrodge 1e156fc
fixed following the PR comments
sacrodge b75c9fe
removed ! from the end of titletext, reflected the same in tests
sacrodge 7180ca0
removed unnecessary logging in the component
sacrodge File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -681,6 +681,10 @@ export const defaultFollowUps: ChatItem = { | |
| pillText: 'Some auto reply', | ||
| prompt: 'Some random auto reply here.', | ||
| }, | ||
| { | ||
| command: Commands.MODIFIED_FILES_DEMO, | ||
| pillText: 'Modified files demo', | ||
| }, | ||
| ], | ||
| }, | ||
| }; | ||
|
|
@@ -2181,7 +2185,7 @@ export const mcpToolRunSampleCardInit:ChatItem = // Summary Card | |
| filePaths: ['Running'], | ||
| details: { | ||
| 'Running': { | ||
| description: 'Work in progress!', | ||
| description: 'Work in progress...', | ||
|
||
| icon: null, | ||
| labelIcon: 'progress', | ||
| labelIconForegroundStatus: 'info', | ||
|
|
||
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
Just curious: Why we need two addChatItem here?
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.
I will remove this and find a way to provide demo differently