Skip to content

feat: add campaign settings modal save action#235

Open
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:codex/campaign-settings-modal-216
Open

feat: add campaign settings modal save action#235
saurabhhhcodes wants to merge 1 commit into
Kuldeeep18:mainfrom
saurabhhhcodes:codex/campaign-settings-modal-216

Conversation

@saurabhhhcodes

Copy link
Copy Markdown
Contributor

Closes #216\n\nAdds an explicit save action to the campaign settings modal, keeps the existing settings fields wired into campaign persistence, and lets the gear icon open a usable settings workflow from the campaign builder top bar.\n\nValidation:\n- git diff --check\n- headless Chrome render check against http://127.0.0.1:4174/campaign-builder.html\n

@Kuldeeep18

Copy link
Copy Markdown
Owner

🚀 PR Guidelines — Read Before Raising a PR

Hey contributors 👋

I’m LeadOrbit's Bot, and I’ll review every PR before it gets merged.

✅ Your PR will only be merged if:

  • The code works correctly and u star the repo
  • There are no unnecessary changes
  • Issues pointed out by CodeRabbit are fixed
  • The PR follows clean coding practices
  • The project structure is maintained

❌ PRs that may be rejected:

  • Copy-paste or AI spam code
  • Unrelated changes
  • Low-quality README edits just for contribution count
  • Ignoring review comments
  • Broken builds or failing checks

Before submitting:

  1. Run and test your code properly
  2. Resolve all CodeRabbit suggestions
  3. Keep your PR focused and clean

And if you find the project useful, consider ⭐ starring the repository — it helps the project grow and motivates further development.

Quality contributions > PR count.

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.

LO-037 [Medium]: Implement Campaign Settings Modal in Campaign Builder

2 participants