feat(settingsInterfaces.ts): setting to skip blocklist modal#3277
feat(settingsInterfaces.ts): setting to skip blocklist modal#3277ofgrenudo wants to merge 5 commits into
Conversation
…gs interace addes skipBlocklistModal to relevant areas
…interface value skipBlocklistM Accommodates behavior for interface value skipBlocklistModal essentially grabs value from settings interface and if its true, it just auto adds to block list. if its false, it sets the block list modal to pop up
…enylisting media adds a few strings in the translation for auto denylisting media with a toggle button
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughAdds a default-off ChangesBlocklist confirmation setting
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant SettingsMain
participant SettingsAPI
participant SettingsContext
participant BlocklistAction
participant BlocklistModal
SettingsMain->>SettingsAPI: Save skipBlocklistModal
SettingsAPI->>SettingsContext: Provide public setting
BlocklistAction->>SettingsContext: Read skipBlocklistModal
alt Enabled
BlocklistAction->>BlocklistAction: Invoke onClickHideItemBtn
else Disabled
BlocklistAction->>BlocklistModal: Open confirmation modal
end
Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
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. Comment |
Description
This PR extends the settings interface to account for a default behavior of auto adding content to the denylist. Once the settings interface was extended I account for the behavior of a present variable where it is true, and if true then we automatically add the content to the deny list. if it is false, we retain the modal that pops up by default.
How Has This Been Tested?
Screenshots / Logs (if applicable)
Video uploaded to google drive, too big to attach here.
https://drive.google.com/file/d/1xysfB8ys-bK-WoPY7i_atiNS51bPfWQ3/view
Checklist:
pnpm buildpnpm i18n:extractAI Discolsure
AI was used to extend the settings interface, and all other places the new interface would be used.
Summary by CodeRabbit
Summary by CodeRabbit
New Features
Bug Fixes