Skip to content

Added content type to user progressed event. - #15

Merged
myluki2000 merged 2 commits into
mainfrom
feature_publishing_content_type_on_progress
Sep 5, 2025
Merged

myluki2000 merged 2 commits into
mainfrom
feature_publishing_content_type_on_progress

Conversation

@pkunz96

@pkunz96 pkunz96 commented Sep 3, 2025 •

Copy link
Copy Markdown
Contributor

Description of changes

Inlcuded the content type 'FLASHCARDS' into the published 'ContentProgressedEvent'.

How has this been tested?

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

  • automated unit test
  • automated integration test
  • [x ] 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

Comment thread build.gradle

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

In der neuen Version sind die Klassen TopicPublisherConfiguration, DevTopicPublisherConfiguration und GraphQlScalarsAndValidationConfiguration in common reingezogen worden
=> Wenn man den service startet beschwert er sich, dass es diese Configurations zweimal gibt
=> Aus dem config Ordner löschen

@myluki2000

Copy link
Copy Markdown
Contributor

In der neuen Version sind die Klassen TopicPublisherConfiguration, DevTopicPublisherConfiguration und GraphQlScalarsAndValidationConfiguration in common reingezogen worden => Wenn man den service startet beschwert er sich, dass es diese Configurations zweimal gibt => Aus dem config Ordner löschen

resolved

@myluki2000
myluki2000 merged commit 6157edf into main Sep 5, 2025
1 check failed
@myluki2000
myluki2000 deleted the feature_publishing_content_type_on_progress branch September 5, 2025 12:52
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