Closed SnehaHS65 closed 1 month ago
This update involves renaming the LetterSoundCorrespondenceContributionEvent
class to LetterSoundContributionEvent
across various files and adjusting method signatures accordingly. This change aims to align with the updated entity naming conventions. Additionally, there are updates to import statements, and SQL schema changes to reflect the new class name.
File Path | Change Summary |
---|---|
src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java |
Renamed methods and class references from LetterSoundCorrespondenceContributionEvent to LetterSoundContributionEvent . |
src/main/java/ai/elimu/dao/jpa/LetterSoundContributionEventDaoJpa.java |
Updated method signatures and implementations to use LetterSoundContributionEvent instead of LetterSoundCorrespondenceContributionEvent . |
src/main/java/ai/elimu/dao/LetterSoundPeerReviewEventDao.java |
Renamed methods and class references from LetterSoundCorrespondenceContributionEvent to LetterSoundContributionEvent . |
src/main/java/ai/elimu/model/contributor/LetterSoundContributionEvent.java |
Renamed class from LetterSoundCorrespondenceContributionEvent to LetterSoundContributionEvent . |
src/main/java/ai/elimu/model/contributor/LetterSoundPeerReviewEvent.java |
Updated method signatures to use LetterSoundContributionEvent instead of LetterSoundCorrespondenceContributionEvent . |
src/main/java/ai/elimu/util/db/DbContentImportHelper.java |
Updated import statements and class instantiation to reflect the renaming of LetterSoundCorrespondenceContributionEvent to LetterSoundContributionEvent . |
src/main/java/ai/elimu/web/content/letter_sound/LetterSoundCreateController.java |
Updated import statements and class instantiation in the handleSubmit method to use LetterSoundContributionEvent instead of LetterSoundCorrespondenceContributionEvent . |
src/main/java/ai/elimu/web/content/letter_sound/LetterSoundEditController.java |
Updated import statements and method usage to reflect the renaming to LetterSoundContributionEvent . |
src/main/java/ai/elimu/web/content/peer_review/LetterSoundPeerReviewEventCreateController.java |
Changed import statements and variable types to use LetterSoundContributionEvent instead of LetterSoundCorrespondenceContributionEvent . |
src/main/resources/META-INF/jpa-schema-export.sql |
Added table creation and foreign key constraints for LetterSoundContributionEvent , and removed the corresponding entries for LetterSoundCorrespondenceContributionEvent . |
src/main/resources/db/migration/2004011.sql |
Introduced changes for renaming a table from LetterSoundContributionEvent to LetterSoundCorrespondenceContributionEvent . |
LetterSoundCorrespondence
to LetterSound
by updating LetterSoundContributionEvent
. The remaining tasks involve renaming the LetterSound
entity and related methods.Common learnings
``` Learnt from: venkatesh2k3 PR: elimu-ai/webapp#1682 File: src/main/java/ai/elimu/dao/LetterSoundContributionEventDao.java:13-13 Timestamp: 2024-07-06T17:37:44.413Z Learning: In the `LetterSoundContributionEventDao.java` file, the method signatures should use `LetterSoundContributionEvent` instead of `LetterSoundCorrespondenceContributionEvent` to maintain consistency with the renaming of entities. ```
Attention: Patch coverage is 22.22222%
with 7 lines
in your changes missing coverage. Please review.
Project coverage is 15.26%. Comparing base (
99f378d
) to head (97a4217
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@jo-elimu , Please review my PR to rename only LetterSoundCorrespondenceContributionEvent -> LetterSoundContributionEvent.
Please do not resolve comments that have not been addressed
I have addressed your comments in the latest commit. Can you please check
@SnehaHS65 Your DB migration script ran successfully in production:
[HIN] 23:46:03.987 [main] INFO ai.elimu.util.db.DbMigrationHelper - performDatabaseMigration
[HIN] 23:46:03.987 [main] INFO ai.elimu.util.db.DbMigrationHelper - pomVersion: 2.4.11
[HIN] 23:46:03.988 [main] INFO ai.elimu.util.db.DbMigrationHelper - pomVersionAsInteger: 2004011
[HIN] 23:46:04.373 [main] INFO ai.elimu.util.db.DbMigrationHelper - dbMigrations.size(): 42
[HIN] 23:46:04.374 [main] INFO ai.elimu.util.db.DbMigrationHelper - versionOfMostRecentMigration: 2004010
[HIN] 23:46:04.374 [main] INFO ai.elimu.util.db.DbMigrationHelper - Looking up pending DB migrations after version 2004010
[HIN] 23:46:04.374 [main] INFO ai.elimu.util.db.DbMigrationHelper - Looking up file "db/migration/2004011.sql"...
[HIN] 23:46:04.374 [main] INFO ai.elimu.util.db.DbMigrationHelper - Migration script found for version 2004011
[HIN] 23:46:04.379 [main] INFO ai.elimu.util.db.DbMigrationHelper - Executing sql: DROP TABLE IF EXISTS `LetterSoundContributionEvent`;
[HIN] 23:46:04.415 [main] INFO ai.elimu.util.db.DbMigrationHelper - Executing sql: ALTER TABLE `LetterSoundCorrespondenceContributionEvent` RENAME `LetterSoundContributionEvent`;
[HIN] 23:46:04.428 [main] INFO ai.elimu.util.db.DbMigrationHelper - Updating current version (2004011)
[HIN] 23:46:04.499 [main] INFO ai.elimu.util.db.DbMigrationHelper - Database migration complete!
refs : #1677 Refactor LetterSoundCorrespondenceContributionEvent -> LetterSoundContributionEvent