Remove unused imports

[uldisrudzitis]

Summary by CodeRabbit

• Refactor
  • Cleaned up unused reactive imports and an unused action reference in regional costs and settings modules to streamline code and reduce potential bundle overhead.
  • Renamed an internal reset handler in shop settings for clearer intent; behavior unchanged.
  • Improves readability and maintainability with no impact on user workflows, regional cost behavior, or settings data.

[coderabbitai]

Walkthrough

Removes unused RxJS and local imports in two shop state files and drops an unused action import. Renames an action handler method in shop-settings-config state from resetProducts to resetShopSettingsConfig without changing logic. No public entity additions; one method name updated.

Changes

Cohort / File(s)	Summary
Import cleanup (RxJS/local) editor/src/app/shop/regional-costs/shop-regional-costs.state.ts, editor/src/app/shop/settings/shop-settings-config.state.ts	Removed unused imports: rxjs concat; rxjs/operators tap, pairwise; local UserState; unused AddShopRegionSiteAction import. No logic changes.
Action handler rename editor/src/app/shop/settings/shop-settings-config.state.ts	Renamed method resetProducts to resetShopSettingsConfig; behavior unchanged (sets state to defaultState). Public method signature updated accordingly.

Sequence Diagram(s)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Fix angular shop issues with multisite usage #584 — Also touches shop state files, removing AddShopRegionSiteAction and adjusting RxJS imports/handler names in a similar manner.

Poem

A tidy hop through fields of code,
I nibbled imports, lightened load.
A name now truer—settings reset—
Same burrow, cleaner silhouette.
Ears up, I stamp: refactor done!
Fewer twigs, the garden’s won. 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The pull request title accurately describes the removal of unused imports, which constitutes the primary change in this changeset, and is concise and clear even though it does not list every minor update such as the method renaming.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-shop-editor-issues-2

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 09ee1cf and 8e7bb1e.

📒 Files selected for processing (2)

• editor/src/app/shop/regional-costs/shop-regional-costs.state.ts (0 hunks)
• editor/src/app/shop/settings/shop-settings-config.state.ts (2 hunks)

💤 Files with no reviewable changes (1)

• editor/src/app/shop/regional-costs/shop-regional-costs.state.ts

🔇 Additional comments (2)

editor/src/app/shop/settings/shop-settings-config.state.ts (2)

1-1: LGTM! Import cleanup is correct.

The remaining RxJS operators are all used in the code, and removing unused imports improves maintainability.

---

61-61: Rename validated; no remaining references
All occurrences of resetProducts are in other state files and unrelated; shop-settings-config.state.ts now consistently uses resetShopSettingsConfig.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}