Skip to content

Conversation

@katinthehatsite
Copy link
Contributor

Related issues

Fixes STU-494

Proposed Changes

Currently, if the user opens the Add site form, chooses a custom WP version that needs to be downloaded and then their connection cuts, Studio attempts to download that WP version and this results in a gibberish error.

I propose to force the latest version for creating a site if the user is offline since this comes with Studio already and should work without connection.

Testing Instructions

  • Pull the changes from this branch
  • Start Studio with npm start
  • Click on Add site in the sidebar
  • Select the WP version from the dropdown that is not latest
  • Keep the form open
  • Cut your internet access
  • Observe that the option in the dropdown gets set to latest
  • Confirm that the site gets created and starts successfully with the latest version
  • You can repeat the same steps on trunk and confirm that you can attempt site creation with a custom WP version which results into gibberish error when the app attempts downloading WordPress

Pre-merge Checklist

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

@katinthehatsite katinthehatsite self-assigned this May 13, 2025
@katinthehatsite katinthehatsite requested a review from a team May 13, 2025 12:28
@katinthehatsite
Copy link
Contributor Author

I left the tooltip as currently is when offline e.g. Changing WP version requires an internet connection because I thought it is fitting but we can refine it if we consider this necessary.

Copy link
Contributor

@bcotrim bcotrim left a comment

Choose a reason for hiding this comment

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

Worked as described, nice catch!

Have you considered treating it as an error and disable the button if the version is not latest and is offline? All of this behavior is an edge case, but I could see a scenario where the connection fails for a short time and we change the version without the user noticing it. Having an error would make it more visible.

@katinthehatsite
Copy link
Contributor Author

Have you considered treating it as an error and disable the button if the version is not latest and is offline? All of this behavior is an edge case, but I could see a scenario where the connection fails for a short time and we change the version without the user noticing it. Having an error would make it more visible.

I am in two minds about this because we sort of block the changing of versions already when editing site instead of allowing the user to trigger the error. Perhaps, a middle ground could be to change the tooltip to something like "You are currently offline and can create a site only with the latest version. Selecting a different version requires an internet connection".

Alternatively, we can also source more opinions. On my end, I don't have a strong preference.

@bcotrim
Copy link
Contributor

bcotrim commented May 13, 2025

I like that suggestion!
Just to confirm that I understood correctly, we would adjust the tooltip as you suggested, and on submission always use the latest version if the user is offline, is that correct?

@katinthehatsite
Copy link
Contributor Author

Just to confirm that I understood correctly, we would adjust the tooltip as you suggested, and on submission always use the latest version if the user is offline, is that correct?

Yeah exactly!

Copy link
Contributor

@gavande1 gavande1 left a comment

Choose a reason for hiding this comment

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

It works as expected. Thanks for the improvement. LGTM 👍

CleanShot 2025-05-13 at 19 02 18@2x

Copy link
Contributor

@wojtekn wojtekn left a comment

Choose a reason for hiding this comment

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

The code change looks clear, works as expected. Nice improvement!

@katinthehatsite katinthehatsite merged commit f38331a into trunk May 14, 2025
8 checks passed
@katinthehatsite katinthehatsite deleted the fix/wp-version-selection-offline-form branch May 14, 2025 09:02
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.

5 participants