Skip to content

feat(studio): Set default site directory location#2710

Open
mikaoelitiana wants to merge 7 commits intoAutomattic:trunkfrom
mikaoelitiana:587-default-location
Open

feat(studio): Set default site directory location#2710
mikaoelitiana wants to merge 7 commits intoAutomattic:trunkfrom
mikaoelitiana:587-default-location

Conversation

@mikaoelitiana
Copy link
Contributor

@mikaoelitiana mikaoelitiana commented Mar 5, 2026

Related issues

How AI was used in this PR

  • Copilot handled code changes for default site directory behavior, plus tests and linting based on repo rules.

Proposed Changes

  • Resolve the default site directory dynamically (dialogs, copy site, proposed path, and site name generation) instead of using a constant because the path is now changed in run time
  • Add user settings support for saving a default site directory and a picker UI.
  • Update tests to cover the new behavior and mocks.

Testing Instructions

  • npm test -- apps/studio/src/tests/ipc-handlers.test.ts
  • npm run lint
  • npm run make-pot

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

Preview

Enregistrement.de.l.ecran.2026-03-05.a.14.33.19.mov

@mikaoelitiana mikaoelitiana changed the title Update default site directory handling feat: Set default site directory location Mar 5, 2026
@mikaoelitiana mikaoelitiana marked this pull request as ready for review March 5, 2026 13:40
@mikaoelitiana mikaoelitiana changed the title feat: Set default site directory location feat(studio): Set default site directory location Mar 5, 2026
@wojtekn
Copy link
Contributor

wojtekn commented Mar 17, 2026

Hi @mikaoelitiana ! Thanks for your contribution. Feature-wise, it works well.

It would be great to adjust field UI to make it consistent with the path selector we have in 'Add site' flow:

CleanShot 2026-03-17 at 11 04 44@2x

Currently, the default path selection is a bit different:

CleanShot 2026-03-17 at 11 06 34@2x

Let me know if you need help with this change.

@wojtekn wojtekn mentioned this pull request Mar 17, 2026
@mikaoelitiana
Copy link
Contributor Author

Hi @mikaoelitiana ! Thanks for your contribution. Feature-wise, it works well.

It would be great to adjust field UI to make it consistent with the path selector we have in 'Add site' flow:

CleanShot 2026-03-17 at 11 04 44@2x

Currently, the default path selection is a bit different:

CleanShot 2026-03-17 at 11 06 34@2x

Let me know if you need help with this change.

Sure I will fix that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Set a default "new site" location

2 participants