Skip to content

Loading states for sync-in-progress #384

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

Merged
merged 17 commits into from
May 23, 2024
Merged

Loading states for sync-in-progress #384

merged 17 commits into from
May 23, 2024

Conversation

e-for-eshaan
Copy link
Contributor

@e-for-eshaan e-for-eshaan commented May 23, 2024

Issues linked

#340

Start

Screen.Recording.2024-05-23.at.9.15.21.AM.online-video-cutter.com.1.mp4

End

Screen.Recording.2024-05-23.at.9.15.21.AM.online-video-cutter.com.mp4
  • Adds the bookmarked_repos field to the Dora Metrics API response. The field contains an array of bookmarked repository IDs.
  • Additionally, a new function getBookmarkedRepos is implemented to fetch the bookmarked repos for a specific team.
  • Loading states for sync-in-progress was made using a hook that polls and watches if repos are synced or not
  • changed fresh-org logic and created a hook to check and poll dora-metrics API to see if repos are synced or not

@e-for-eshaan e-for-eshaan changed the title "Add bookmarked repos to Dora Metrics API response" Loading states for sync-in-progress May 23, 2024
@e-for-eshaan e-for-eshaan requested a review from shivam-bit May 23, 2024 03:55

endpoint.handle.GET(nullSchema, async (req, res) => {
if (req.meta?.features?.use_mock_data) {
return res.send([1, 2, 3].map(() => uuid()));
Copy link
Contributor

Choose a reason for hiding this comment

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

[1,2,3] seems very odd, maybe directly use [uuid(),uuid(),uuid()] or declare it inside a mock file

@e-for-eshaan e-for-eshaan merged commit 69fc3cc into main May 23, 2024
1 check passed
@e-for-eshaan e-for-eshaan deleted the fix/repo-sync-loader branch May 23, 2024 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants