Adds project refresh functionality to context and UI #605
Conversation
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
Introduces a `refreshProjects` method to the ProjectsContext, enabling manual and programmatic refresh of project data. Updates the ProjectsContextProvider to include this method and refactors the initial refresh logic. Integrates the refresh functionality into the ProjectListItem component to trigger a refresh when a project is selected. Also includes minor code style improvements for consistency.
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds project refresh functionality by introducing a refreshProjects method to the ProjectsContext and integrating it into the UI. The changes enable manual project data refresh when projects are selected and include code style improvements for consistency.
- Added
refreshProjectsmethod to ProjectsContext interface and provider - Integrated refresh functionality into ProjectListItem to trigger on project selection
- Applied consistent code formatting with semicolons and proper JSX formatting
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/common/context/ProjectsContext.ts | Added refreshProjects method to context interface and default value |
| src/features/projects/view/ProjectsContextProvider.tsx | Exposed refreshProjects method and refactored initial refresh logic |
| src/features/sidebar/view/internal/sidebar/projects/ProjectListItem.tsx | Integrated refresh functionality and applied code style improvements |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Simplifies the useEffect dependency array by removing the unneeded dependency on the refreshProjects function. This prevents unnecessary re-renders and ensures the effect runs only once, improving performance and maintaining expected behavior. No functional changes introduced.
This reverts commit 5733e60.
ulrikandersen
approved these changes
Oct 1, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
Introduces a
refreshProjectsmethod to the ProjectsContext, enabling manual and programmatic refresh of project data. Updates the ProjectsContextProvider to include this method and refactors the initial refresh logic.Integrates the refresh functionality into the ProjectListItem component to trigger a refresh when a project is selected. Also includes minor code style improvements for consistency.
Description
Motivation and Context
Screenshots (if appropriate):
Types of changes