Skip to content

fix(ui): filtering repos by active status#2071

Merged
liangfung merged 5 commits intomainfrom
feat-ui-filtering-repos-by-active-status
May 8, 2024
Merged

fix(ui): filtering repos by active status#2071
liangfung merged 5 commits intomainfrom
feat-ui-filtering-repos-by-active-status

Conversation

@liangfung
Copy link
Copy Markdown
Collaborator

liangfung added 2 commits May 8, 2024 15:34
relay

feat(ui): filtering repos by active status
@liangfung liangfung requested a review from wsxiaoys May 8, 2024 07:44
@liangfung liangfung marked this pull request as ready for review May 8, 2024 07:44
@wsxiaoys wsxiaoys requested a review from wwayne May 8, 2024 08:33
isAllLoaded: isInactiveRepositoriesLoaded
} = useAllInactiveRepositories(providerId)

const doFetchRepositories = (
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe the name can just be fetchRepositories?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed

})
}

const doLoadPage = async (pageNo: number) => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

loadPage ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

renamed

@liangfung liangfung merged commit dfb4ca3 into main May 8, 2024
@liangfung liangfung deleted the feat-ui-filtering-repos-by-active-status branch May 8, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants