Skip to content

Add calculator Datadog app - #2

Closed
setnilson wants to merge 1 commit into
mainfrom
add-calculator-app-202608181943
Closed

setnilson wants to merge 1 commit into
mainfrom
add-calculator-app-202608181943

Conversation

@setnilson

Copy link
Copy Markdown
Owner

Summary

Builds the requested calculator experience as a new Datadog App workspace at apps/sarah-s-app-tue-aug-18-3-34-53-pm so it can be developed, built, and uploaded independently.

Changes

  • Scaffolded apps/sarah-s-app-tue-aug-18-3-34-53-pm as a Vite React Datadog App.
  • Replaced the default welcome screen with a responsive calculator supporting digits, decimal input, sign toggle, percent, delete, clear, chained operators, equals, and divide-by-zero error handling.
  • Added scoped calculator styling and fixed app metadata/site resolution in vite.config.ts to use the workspace Datadog site fallback.
  • Regenerated the root package-lock.json so the new workspace package and node_modules workspace link are represented.

Testing/Validation

  • npm install --package-lock-only --ignore-scripts
  • npm ci --ignore-scripts
  • Verified package-lock.json contains apps/sarah-s-app-tue-aug-18-3-34-53-pm and node_modules/sarah-s-app-tue-aug-18-3-34-53-pm workspace link entries.
  • npm run typecheck -w apps/sarah-s-app-tue-aug-18-3-34-53-pm
  • npm run lint -w apps/sarah-s-app-tue-aug-18-3-34-53-pm
  • npm run build -w apps/sarah-s-app-tue-aug-18-3-34-53-pm
  • Ran the Vite dev server and used Playwright to assert 12 + 7 = 19, then captured desktop and mobile screenshots.

PR by Bits - View session in Datadog

Comment @datadog-staging to request changes

Co-authored-by: setnilson <110850377+setnilson@users.noreply.github.com>
@datadog-staging

datadog-staging Bot commented Aug 18, 2026

Copy link
Copy Markdown

View session in Datadog

Bits Code status: ✅ Done

Comment @datadog-staging to request changes

@datadog-staging

Copy link
Copy Markdown

I can only run on private repositories.

@setnilson setnilson closed this Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants