Skip to content

Fix LC invite and IG filtering issues#3056

Merged
DevWithPranav merged 4 commits into
feat/campusfrom
dev-server
Jul 21, 2026
Merged

Fix LC invite and IG filtering issues#3056
DevWithPranav merged 4 commits into
feat/campusfrom
dev-server

Conversation

@DevWithPranav

Copy link
Copy Markdown
Collaborator

No description provided.

@DevWithPranav
DevWithPranav merged commit 40dffba into feat/campus Jul 21, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Triage to QA in µLearn Delivery Jul 21, 2026
@greptile-apps

greptile-apps Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes learning-circle invitation handling and filters interest groups by status. The main changes are:

  • Accept link_id from either the request URL or body.
  • Add tests for invitation acceptance, rejection, and invalid identifiers.
  • Limit interest-group list and detail responses to active groups.
  • Add the creator MUID to project responses.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.

Important Files Changed

Filename Overview
api/common/common_views.py Filters public interest-group list and detail queries to active groups.
api/dashboard/ig/dash_ig_view.py Filters the dashboard interest-group list to active groups.
api/dashboard/learningcircle/learningcircle_views.py Supports invitation identifiers from both URL parameters and request bodies.
api/dashboard/learningcircle/tests/test_learningcircle.py Tests both invitation request forms and their main error cases.
api/dashboard/projects/projects_serializer.py Adds the project creator's MUID to serialized responses.
utils/types.py Defines shared interest-group status values.

Reviews (1): Last reviewed commit: "Merge pull request #3055 from gtech-mule..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: QA

Development

Successfully merging this pull request may close these issues.

3 participants