Skip to content

[DEV-570] added study group data integration - #120

Merged
ericguo202 merged 3 commits into
previewfrom
study-group-page-updated
Sep 8, 2026
Merged

ericguo202 merged 3 commits into
previewfrom
study-group-page-updated

Conversation

@adambelouad

Copy link
Copy Markdown
Member

References

Proposed Changes

  • shows study group cards from getAllStudyGroups

@linear

linear Bot commented Apr 15, 2026

Copy link
Copy Markdown

@vercel

vercel Bot commented Apr 15, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
help Ready Ready Preview Sep 8, 2026 3:21am UTC

Request Review

Comment thread frontend/app/study-groups/page.tsx Outdated
// Placeholder for button click action
};
useEffect(() => {
getAllStudyGroup().then(setStudyGroups);

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.

This should prob be wrapped in an async

@issacli-0821

Copy link
Copy Markdown
Contributor

Also should resolve the lint warnings on the PR!

@ericguo202
ericguo202 self-requested a review September 8, 2026 03:20

@ericguo202 ericguo202 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Linter errors have been fixed

@ericguo202
ericguo202 merged commit beb8021 into preview Sep 8, 2026
4 of 5 checks passed
@ericguo202
ericguo202 deleted the study-group-page-updated branch October 11, 2026 17:32

This branch was successfully deployed

1 active deployment
Preview — 7810443a Deployed Sep 8, 2026 by vercel[bot]
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.

3 participants