Closed jo-elimu closed 3 weeks ago
The overall change involves removing the platform
field and related functionalities from the PeerReviewEvent
class and its usages across the project. This includes modifications to multiple controllers where the platform
was being set, as well as updates to database schema files to reflect the removal of the platform
column.
File Path | Change Summary |
---|---|
src/.../PeerReviewEvent.java |
Removed Platform enum-related fields and methods. |
src/.../AudioPeerReviewEventCreateController.java |
Removed setting of the Platform enum for audioPeerReviewEvent . |
src/.../LetterSoundPeerReviewEventCreateController.java |
Removed setting of the Platform enum for letterSoundPeerReviewEvent . |
src/.../NumberPeerReviewEventCreateController.java |
Removed setting of the platform field for numberPeerReviewEvent . |
src/.../StoryBookPeerReviewEventCreateController.java |
Removed setting of the Platform enum for storyBookPeerReviewEvent . |
src/.../WordPeerReviewEventCreateController.java |
Removed setting of the Platform enum for wordPeerReviewEvent . |
src/.../jpa-schema-export.sql |
Removed platform column declarations from multiple tables. |
src/.../db/migration/2004005.sql |
New file for dropping the platform column from multiple tables. |
src/main/java/ai/elimu/web/content/peer_review/StoryBookPeerReviewEventCreateController.java (1)
Line range hint `29-29`: **Code change is correct and consistent with refactoring objectives.** The removal of the line setting the platform field aligns with the refactoring objectives. No issues found.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 15.24%. Comparing base (
6a2611f
) to head (e6b6c1b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
refs #1674