Skip to content

OUT-2269 | Client Home app: Save Changes Bar shows up once I install "Client home" app - #100

Merged
arpandhakal merged 4 commits into
mainfrom
OUT-2269
Aug 29, 2025
Merged

OUT-2269 | Client Home app: Save Changes Bar shows up once I install "Client home" app#100
arpandhakal merged 4 commits into
mainfrom
OUT-2269

Conversation

@arpandhakal

Copy link
Copy Markdown
Collaborator

The main changes are

  • added prepareCustomLabel for dif comparison while triggering save changes.
  • applied prepareCustomLabel on defaultState while checking for changes diff.

- added prepareCustomLabel for dif comparison while triggering save changes.
@arpandhakal
arpandhakal requested a review from rrojan August 29, 2025 07:11
@arpandhakal arpandhakal self-assigned this Aug 29, 2025
@linear

linear Bot commented Aug 29, 2025

Copy link
Copy Markdown

@vercel

vercel Bot commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
client-home Ready Ready Preview Comment Aug 29, 2025 9:20am

Comment thread src/app/components/EditorInterface.tsx Outdated
appState?.appState.originalTemplate?.replace(/\s/g, '') !==
defaultState.replace(/\s/g, '') ||
prepareCustomLabel(
defaultState.replace(/\s/g, ''),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Use modern replaceAll syntax instead

Comment thread src/app/components/EditorInterface.tsx Outdated

@rrojan rrojan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 🏆

@arpandhakal
arpandhakal merged commit c36263b into main Aug 29, 2025
3 checks passed
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