Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(coredata): updating coredata and creating initalizers #395

Merged
merged 8 commits into from
Feb 17, 2023

Conversation

bassrock
Copy link
Member

@bassrock bassrock commented Feb 15, 2023

Summary

  • Update Item, Slate, SlateLineup, Recomendation to make certain coredata fields non-null
  • Create new initializer files using a Swift hack on initializers. (thanks swift and coredata ๐Ÿ˜ฎโ€๐Ÿ’จ )
  • Update all uses of optional fields
  • Remove convenience new initializer for core data. So we use the custom inits.

References

  • Links to docs, tickets, designs if available

Implementation Details

  • Overview of work that was implemented and changes made to the codebase

Test Steps

  • Write out what QA should do to verify this change works as expected and hasn't introduced regressions. Can mention specific OS versions, devices, areas of the app to test as needed.

PR Checklist:

  • Added Unit / UI tests
  • Self Review (review, clean up, documentation, run tests)
  • Basic Self QA

Screenshots

@bassrock bassrock marked this pull request as ready for review February 16, 2023 23:34
@bassrock bassrock requested a review from a team as a code owner February 16, 2023 23:34
@bassrock bassrock force-pushed the non-null-fields branch 2 times, most recently from 816bdf8 to 1c260f6 Compare February 17, 2023 19:15
@pocket-ci
Copy link
Contributor

pocket-ci commented Feb 17, 2023

Messages
๐Ÿ“– No SwiftLint violations! ๐ŸŽ‰
๐Ÿ“– Project coverage: 75.82%
๐Ÿ“– Edited 30 files
๐Ÿ“– Created 11 files

PocketKit: Coverage: 75.07

File Coverage
SlateDetailViewModel.swift 93.48% โœ…
HomeViewModel.swift 86.79% โœ…
SavedItem+ItemsListItem.swift 100.0% โœ…
HomeRecommendationCellViewModel.swift 100.0% โœ…

Sync: Coverage: 89.4

File Coverage
SavedItem+RemoteMapping.swift 90.47% โœ…
Recommendation+CoreDataProperties.swift 100.0% โœ…
ArchiveService.swift 96.66% โœ…
Space.swift 84.58% โœ…
PocketSource.swift 87.44% โœ…
SavedItem+CoreDataClass.swift 60.0% โœ…
Slate+CoreDataProperties.swift 100.0% โœ…
SlateLineup+CoreDataProperties.swift 100.0% โœ…
Slate+CoreDataClass.swift 43.75% โš ๏ธ
Item+CoreDataProperties.swift 100.0% โœ…
SlateLineup+CoreDataClass.swift 43.75% โš ๏ธ
Item+remoteMapping.swift 91.62% โœ…
Item+CoreDataClass.swift 60.0% โœ…
SlateService.swift 83.02% โœ…
SlateLineup+remoteMapping.swift 91.94% โœ…
SavedItem+CoreDataProperties.swift 100.0% โœ…
Recommendation+CoreDataClass.swift 35.71% โš ๏ธ
PocketSaveService.swift 92.09% โœ…
FetchList.swift 98.78% โœ…

Generated by ๐Ÿšซ Danger Swift against bc25116

Copy link
Contributor

@cyndichin cyndichin left a comment

Choose a reason for hiding this comment

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

on a high level looks good, didn't do a thorough review. We can fix issues if we encounter them.

@bassrock bassrock merged commit 1952903 into develop Feb 17, 2023
@bassrock bassrock deleted the non-null-fields branch February 17, 2023 20:31
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