Skip to content

feat: add SMM service - #21

Merged
mrjvs merged 3 commits into
masterfrom
feat/smm-course-delete
Aug 1, 2026
Merged

feat: add SMM service#21
mrjvs merged 3 commits into
masterfrom
feat/smm-course-delete

Conversation

@mrjvs

@mrjvs mrjvs commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Changes:

  • Added SMM service
  • Added method to delete a course in SMM, to be used by the admin panel so mods can delete rule-breaking courses
  • Bumped version

@mrjvs
mrjvs requested a review from DaniElectra July 31, 2026 20:24
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

The latest Buf updates on your PR. Results from workflow Validate Pull Request / check_buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped✅ passed✅ passedAug 1, 2026, 12:25 PM

@DaniElectra DaniElectra 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.

Ideally in the future this should be implemented as a generic DataStore grpc service, possibly on #5 . I won't make that a blocker for this though

Comment thread protobufs/smm/v1/smm_service.proto
@mrjvs
mrjvs requested a review from DaniElectra August 1, 2026 09:24
@mrjvs
mrjvs requested review from DaniElectra and removed request for DaniElectra August 1, 2026 12:25

@DaniElectra DaniElectra 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.

LGTM

@mrjvs
mrjvs merged commit 01b8423 into master Aug 1, 2026
3 checks passed
@mrjvs
mrjvs deleted the feat/smm-course-delete branch August 1, 2026 12:28
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