Skip to content

Remove unused ComponentStore#1023

Merged
floehopper merged 1 commit intomainfrom
remove-unused-component-store
May 28, 2024
Merged

Remove unused ComponentStore#1023
floehopper merged 1 commit intomainfrom
remove-unused-component-store

Conversation

@floehopper
Copy link
Contributor

This was introduced in #96, but never used. I suspect the import store from "./app/store" statement in src/web-component.js was intended to import it, but it used the wrong filename.

ComponentStore was later effectively replaced by WebComponentStore in #728.

@floehopper floehopper force-pushed the remove-unused-component-store branch from 69bf0a2 to 58f6d15 Compare May 28, 2024 09:04
@floehopper floehopper temporarily deployed to previews/remove-unused-component-store May 28, 2024 09:04 — with GitHub Actions Inactive
@github-actions
Copy link

@github-actions
Copy link

chrisroos
chrisroos previously approved these changes May 28, 2024
Copy link

@chrisroos chrisroos left a comment

Choose a reason for hiding this comment

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

LGTM 👍

This was introduced in #96, but never used. I suspect the `import store
from "./app/store"` statement in `src/web-component.js` was intended to
import it, but it used the wrong filename.

`ComponentStore` was later effectively replaced by `WebComponentStore`
in #728.
@github-actions
Copy link

@floehopper floehopper merged commit ca69c29 into main May 28, 2024
@floehopper floehopper deleted the remove-unused-component-store branch May 28, 2024 15:19
@floehopper floehopper mentioned this pull request Jun 3, 2024
floehopper added a commit that referenced this pull request Jun 4, 2024
### Added

- Add `project_name_editable` attribute to web component (#1009)
- Fires custom event when the theme changes (#1015)
- Add `output_only` attribute to web component (#1019 & originally #782)
- Add `assets_identifier` attribute to web component (#1019 & originally
#901)
- Enhance `code` attribute on web component to override project main
component content (#1019 & originally #901)
- Add `runCode`, `stopCode` & `rerunCode` methods to web component
(#1019 & originally #899)
- Send error details in "editor-runCompleted" event (#1019 & originally
#915)
- Return error details to web component (#1019 & originally #915)
- Add `output_panels` attribute to web component (#1019 & originally
#909)

### Changed

- Remove unused `/embedded/projects/:identifier` route (#1013)

### Fixed

- Remove unused `REACT_APP_LOGIN_ENABLED` env var (#1006)
- Fix infinite remix loop when `BYPASS_AUTH` set in `editor-api` (#1007)
- Fixes for docker-compose.yml (#1008)
- Fix deprecation warnings in GitHub Actions (#1011)
- Removed unused `isEmbedded` param from `useProject` call in
`EmbeddedViewer` (#1016)
- Improvements to Cypress specs in CI (#1017)
- Fix warnings and verbose output when starting Webpack Dev Server
(#1018)
- Add e2e spec for project remix behaviour in web component (#1020)
- Fix initial value of `user` in `WebComponentLoader` (#1021)
- Make `authKey` in e2e web component spec more realistic (#1022)
- Remove unused `ComponentStore` (#1023)
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.

2 participants