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(edit-content) refactor to use signal store as store Edit Content #30201

Merged
merged 6 commits into from
Oct 8, 2024

Conversation

oidacra
Copy link
Member

@oidacra oidacra commented Oct 1, 2024

Proposed Changes

  • refactor to use signal store as store Edit Content

Checklist

  • Tests
  • Translations
  • Security Implications Contemplated (add notes if applicable)

Additional Info

** any additional useful context or info **

This PR fixes: #30020

@oidacra oidacra marked this pull request as ready for review October 2, 2024 01:13
@oidacra oidacra linked an issue Oct 2, 2024 that may be closed by this pull request
@oidacra oidacra force-pushed the issue-30020-migration-signal-store branch from daa4bc7 to adf50ba Compare October 2, 2024 01:24
@oidacra oidacra force-pushed the issue-30020-migration-signal-store branch from ee32b38 to e8f99f0 Compare October 2, 2024 20:03
@oidacra oidacra force-pushed the issue-30020-migration-signal-store branch from e8f99f0 to 733a094 Compare October 7, 2024 22:02
@oidacra oidacra requested a review from nicobytes October 7, 2024 22:03
@oidacra oidacra enabled auto-merge October 8, 2024 00:04
@oidacra oidacra added this pull request to the merge queue Oct 8, 2024
Merged via the queue into main with commit 358e597 Oct 8, 2024
21 checks passed
@oidacra oidacra deleted the issue-30020-migration-signal-store branch October 8, 2024 03:22
spbolton pushed a commit that referenced this pull request Nov 11, 2024
…#30201)

### Proposed Changes
* refactor to use signal store as store Edit Content

### Checklist
- [x] Tests
- [ ] Translations
- [ ] Security Implications Contemplated (add notes if applicable)

### Additional Info
** any additional useful context or info **
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.

Migrate Current Store Service to Signal Store
4 participants