Skip to content

Gamification service implementation for ai tutor - #25

Merged
MP-DG merged 24 commits into
mainfrom
gamification-service-implementation-for-ai-tutor
Dec 17, 2025
Merged

MP-DG merged 24 commits into
mainfrom
gamification-service-implementation-for-ai-tutor

Conversation

@MP-DG

@MP-DG MP-DG commented Dec 15, 2025

Copy link
Copy Markdown
Contributor

Description of changes

Added more features to the tutor. Allows

  • proactive feedback when an assignment is completed
  • saves user specific information about PlayerType and SkillLevel
  • individualises prompts according to PlayerType and SkillLevel
  • allows tutor to give feedback to code written by the user
  • changed prompts accordingly

How has this been tested?

Please describe the test strategy you followed. If not tested, please explain why.

  • automated unit test
  • automated integration test
  • manual, exploratory test

In case of manual test, please document the test well including a set of user instructions and prerequisites. Each including an action, it's result, and where appropriate a screenshot.

Checklist before requesting a review

  • My code is easy to understand
  • My code follows the coding guidelines of this project
  • My code fulfills all acceptance criteria
  • The test coverage (line and branch) is reasonably high, especially on Service classes and other classes with complex logic
  • I have made corresponding changes to the documentation or
    the wiki/adr
  • I made no breaking changes in the database schema or if so, I will perform a database migration

Checklist for reviewer

  • The code is easy to understand
  • The code follows
    the coding guidelines of
    this project
  • The code is tested or if not, the reason is documented or discussed
  • The added and existing tests reasonably cover the code change
  • The code has no breaking changes in the database schema or if so, the assignee is aware of it and
    will perform a database migration

@c-neli-r c-neli-r left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Für mich macht der Code einen sehr aufgeräumten und gut strukturierten Eindruck. Ich bin inhaltlich allerdings nicht wirklich in deiner Bachelorarbeit drin, und für die vielen Änderungen war die Zeit zum Durchschauen etwas knapp. Wenn aber alles sauber läuft, würde ich dir mein Approve geben. :)

@ZiyaSelim ZiyaSelim left a comment

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.

Sehr schöner Code und sehr cool dass du des in den bestehenden Code eingearbeitet hast 😄

Comment thread build.gradle
implementation 'de.unistuttgart.iste.meitrex:content_service:1.0.9'
implementation 'com.fasterxml.jackson.module:jackson-module-jsonSchema:2.17.1'
implementation 'io.dapr:dapr-sdk-springboot:1.12.0'
// implementation 'de.unistuttgart.iste.meitrex:gamification_service:1.0.0'

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.

Soll das auskommentiert sein?

@@ -1,3 +1,13 @@
type Query {
_empty: String

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.

Du kannst des _empty dann überall löschen. Des war nur dafür da, dass sich des nicht über ein leeres/nicht vorhandenes Query Objekt beschwert :)

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.

Also TutorController Zeilen 47-50

@MP-DG
MP-DG merged commit 59aeffd into main Dec 17, 2025
3 checks passed
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