-
Notifications
You must be signed in to change notification settings - Fork 78
feat(FR-1878): Show project name in folder explorer for cross-project folders #4961
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: feat_FR-1871_/add-direct-access-to-model-card-modal-header
Are you sure you want to change the base?
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has required the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Coverage report for
|
St.❔ |
Category | Percentage | Covered / Total |
|---|---|---|---|
| 🔴 | Statements | 4.91% | 589/11992 |
| 🔴 | Branches | 4.47% (-0% 🔻) |
377/8433 |
| 🔴 | Functions | 2.75% | 101/3676 |
| 🔴 | Lines | 4.74% | 555/11711 |
Test suite run success
173 tests passing in 13 suites.
Report generated by 🧪jest coverage report action from 2f4a4d3
27db591 to
d1a9bca
Compare
b76d2d3 to
0b2f1eb
Compare
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 adds a "Switch Project" button to the folder explorer when viewing a folder that belongs to a different project than the currently active one. Previously, users would only see a warning message without the ability to easily switch projects, requiring them to manually navigate away and change projects through other UI elements.
Key changes:
- Created a new
FolderProjectMismatchAlertcomponent that displays the warning with an actionable "Switch Project" button - Added comprehensive translations for the new button and messages across all 21 supported languages
- Added a tooltip to the folder view icon in the model card modal for better discoverability
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
react/src/components/FolderProjectMismatchAlert.tsx |
New component that displays a project mismatch alert with a "Switch Project" button |
react/src/components/FolderExplorerModal.tsx |
Updated to use the new FolderProjectMismatchAlert component instead of a simple alert |
react/src/components/ModelCardModal.tsx |
Added tooltip to the folder view icon for better UX |
resources/i18n/en.json |
Added English translations for "SwitchProject", "SwitchedToProject", "FailedToSwitchProject", and "ViewInFolderExplorer" |
resources/i18n/ko.json |
Added Korean translations |
resources/i18n/ja.json |
Added Japanese translations |
resources/i18n/zh-CN.json |
Added Simplified Chinese translations |
resources/i18n/zh-TW.json |
Added Traditional Chinese translations |
resources/i18n/de.json |
Added German translations |
resources/i18n/es.json |
Added Spanish translations |
resources/i18n/fr.json |
Added French translations |
resources/i18n/it.json |
Added Italian translations |
resources/i18n/pt.json |
Added Portuguese translations |
resources/i18n/pt-BR.json |
Added Brazilian Portuguese translations |
resources/i18n/ru.json |
Added Russian translations |
resources/i18n/pl.json |
Added Polish translations |
resources/i18n/tr.json |
Added Turkish translations |
resources/i18n/vi.json |
Added Vietnamese translations |
resources/i18n/th.json |
Added Thai translations |
resources/i18n/ms.json |
Added Malay translations |
resources/i18n/mn.json |
Added Mongolian translations |
resources/i18n/el.json |
Added Greek translations |
resources/i18n/fi.json |
Added Finnish translations |
resources/i18n/id.json |
Added Indonesian translations |
2175d6b to
e2ec60b
Compare
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
Copilot reviewed 24 out of 24 changed files in this pull request and generated 1 comment.
0b2f1eb to
889c8ec
Compare
3f3d674 to
538ba51
Compare
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
Copilot reviewed 22 out of 22 changed files in this pull request and generated 4 comments.
538ba51 to
d67125b
Compare
889c8ec to
a264738
Compare
d67125b to
553c366
Compare
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
Copilot reviewed 21 out of 21 changed files in this pull request and generated 1 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.
ironAiken2
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.
please check copilot's review. It appears that the translation file is missing.
a264738 to
b07f323
Compare
553c366 to
2f4a4d3
Compare
;; my extension is strange. I applied it. |
ironAiken2
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.
It seems we need to modify NotInProject to accept variable assignment :)

Resolves #4960 (FR-1878)
Enhance Folder Explorer with Project Name Display and Alert Type Change
This PR enhances the folder explorer modal to display the specific project name when viewing folders from different projects. Previously, users would only see a generic warning message stating "This folder belongs to a different project." Now, the message includes the actual project name: "This folder belongs to '[project name]' project."
Key changes:
group_nameto the GraphQL query to fetch the project nameThese changes improve the user experience by providing more specific information about folder ownership, making it easier for users to understand which project a folder belongs to when navigating across different projects.