Improve database handling of model indices #7

New Issue

MahmoudEmad · 2025-05-27T18:18:17Z

MahmoudEmad commented

2025-05-27 18:18:17 +00:00

Task

Ensure all model types always have a primary key index entry.
Simplify get_all() implementation by relying on primary key index.
Remove redundant code in get_all() for finding all object IDs.
Improve error handling and clarity in database operations.

### Task - Ensure all model types always have a primary key index entry. - Simplify `get_all()` implementation by relying on primary key index. - Remove redundant code in `get_all()` for finding all object IDs. - Improve error handling and clarity in database operations.

MahmoudEmad self-assigned this 2025-05-27 18:18:28 +00:00

MahmoudEmad referenced this issue

2025-05-27 18:18:57 +00:00

feat: Improve database handling of model indices #6

MahmoudEmad commented

2025-05-27 18:19:34 +00:00

This PR implements this issue

This [PR](https://git.ourworld.tf/herocode/db/pulls/6) implements this issue

MahmoudEmad commented

2025-05-28 08:56:00 +00:00

Issue Update

Fixed an issue in the calendar example where missing object fields caused errors. The problem was identified and corrected. After the fix, the tests ran successfully without any issues.

Changes

Added missing object fields in the calendar example
Verified the fix by rerunning the tests successfully

### Issue Update Fixed an issue in the calendar example where missing object fields caused errors. The problem was identified and corrected. After the fix, the tests ran successfully without any issues. ### Changes * Added missing object fields in the calendar example * Verified the fix by rerunning the tests successfully

Screenshot 2025-05-28 at 11.49.08 AM.png

94 KiB

despiegk closed this issue

2025-07-30 07:37:56 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: herocode/db#7