Skip to content

feat(pinia-orm): Shared hydration cache & pinia actions triggering repo actions#1066

Merged
CodeDredd merged 1 commit intonext-releasefrom
action-enhancements
Mar 15, 2023
Merged

feat(pinia-orm): Shared hydration cache & pinia actions triggering repo actions#1066
CodeDredd merged 1 commit intonext-releasefrom
action-enhancements

Conversation

@CodeDredd
Copy link
Copy Markdown
Owner

@CodeDredd CodeDredd commented Mar 13, 2023

🔗 Linked issue

closes #1058

❓ Type of change

  • 📖 Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • 👌 Enhancement (improving an existing functionality like performance)
  • ✨ New feature (a non-breaking change that adds functionality)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

  1. This is adding a shared hydration cache to improve performance accross repositories.
  2. The pinia actions if called directly now also trigger the store actions so hooks & hydration cache is updated

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@CodeDredd CodeDredd added the enhancement New feature or request label Mar 13, 2023
@CodeDredd CodeDredd changed the base branch from master to next-release March 13, 2023 14:00
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Patch coverage: 72.09% and project coverage change: -0.14 ⚠️

Comparison is base (5cf229e) 99.55% compared to head (9822c03) 99.41%.

Additional details and impacted files
@@               Coverage Diff                @@
##           next-release    #1066      +/-   ##
================================================
- Coverage         99.55%   99.41%   -0.14%     
================================================
  Files                87       88       +1     
  Lines              5826     5854      +28     
  Branches            515      531      +16     
================================================
+ Hits               5800     5820      +20     
- Misses               23       31       +8     
  Partials              3        3              
Impacted Files Coverage Δ
...kages/pinia-orm/src/composables/useStoreActions.ts 82.08% <60.00%> (-17.92%) ⬇️
...es/pinia-orm/src/cache/SharedHydratedDatakCache.ts 100.00% <100.00%> (ø)
packages/pinia-orm/src/composables/useDataStore.ts 100.00% <100.00%> (ø)
packages/pinia-orm/src/composables/useRepo.ts 100.00% <100.00%> (ø)
packages/pinia-orm/src/query/Query.ts 99.51% <100.00%> (ø)
packages/pinia-orm/src/repository/Repository.ts 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@CodeDredd CodeDredd merged commit 58e446e into next-release Mar 15, 2023
@CodeDredd CodeDredd deleted the action-enhancements branch March 15, 2023 19:08
@CodeDredd CodeDredd mentioned this pull request Mar 15, 2023
CodeDredd added a commit that referenced this pull request Mar 22, 2023
* feat(pina-orm): Add `morphToMany` relation (#1060)

* feat(pina-orm): Add `morphToMany` relation

* refactor: change size check

* refactor: correct comments

* fix(pinia-orm): Types in hooks were conflicting (#1074)

closes #1067

* feat(pinia-orm): Shared hydration cache & pinia actions trigger repo actions (#1066)

closes #1058

* test(pinia-orm): Add `undefined` as value

* test(pinia-orm): add missing store action tests

* docs(pinia-orm): remove not correct await (#1087)

closes #1049

* docs: change plausible url (#1095)

* chore: update twitter account

* chore: remove twitter

* chore: Add sponsorkit (#1097)

* chore: add sponsorkit

* chore: add sponsorkit

* docs: Add "what is pinia orm"

* docs: update yarn.lock

* docs: Add new background logo

* feat(nuxt): Add improvements (#1100)

* feat(nuxt): Improvements
- Add autoimports
- Add missing config options
- Use nuxt kit resolver instead of node resolve
- Update build config
- Add nuxt devtools support

* chore(nuxt): fix linting ci

* docs: resolve conflict
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Models from mapped repo instance are not reactive under certain conditions

2 participants