Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove use of page= query param for DevTools embeddings #7405

Open
kenzieschmoll opened this issue May 7, 2024 · 6 comments
Open

Remove use of page= query param for DevTools embeddings #7405

kenzieschmoll opened this issue May 7, 2024 · 6 comments
Labels
devtools-integration Related to the integration with Dart DevTools

Comments

@kenzieschmoll
Copy link
Member

There is a TODO in the DevTools codebase to stop using the page query parameter from IDEs.

@helin24 can you confirm that we do not use this from IntelliJ? If we do not use this, this issue can be closed. If we do, we should migrate to use the path based routing, which has been the DevTools standard for a number of years now.

@kenzieschmoll kenzieschmoll added the devtools-integration Related to the integration with Dart DevTools label May 7, 2024
@helin24
Copy link
Member

helin24 commented May 7, 2024

We do use the page query param, e.g. page=inspector. Is there an alternative way to specify the page?

@helin24
Copy link
Member

helin24 commented May 7, 2024

Ooh actually, looking closer I see we're using /inspector instead. But only after flutter v3.3.0. Is that old enough we can remove? This is sort of a larger question of SDK deprecation.

@kenzieschmoll
Copy link
Member Author

kenzieschmoll commented May 7, 2024

CC @jwren @DanTup - do we have guidelines for how long to leave legacy code before removing?

@helin24
Copy link
Member

helin24 commented May 7, 2024

We have a couple docs on this, but I'm not sure if it's appropriate to post here. I'll send to you directly for now.

@DanTup
Copy link
Contributor

DanTup commented May 8, 2024

CC @jwren @DanTup - do we have guidelines for how long to leave legacy code before removing?

There are probably two parts to this question... How long to leave code in the SDK to support old versions of IDE plugins, and how long to leave code in IDE plugins to support old versions of the SDK. It's possible the answer is different for each of these (in part because upgrading SDK is often a much bigger job than upgrading IDE plugins and IDE plugins may frequently need to be updated to handle changes in SDKs).

For &page= specifically, Dart-Code stopped using it for new SDKs approximately 2 years ago (Dart-Code/Dart-Code#3800). I'd be surprised if a 2 year old version of Dart-Code works against modern SDKs because of other changes (for ex. changes like the pub binary being removed and the legacy debug adapters not being updated) so I don't think you need to worry about Dart-Code here.

@helin24
Copy link
Member

helin24 commented May 8, 2024

I don't think we've thought much about the first issue (how long to leave code in the SDK to support old plugins) because of what you mentioned, that we frequently tell users to update plugins if they're having a problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devtools-integration Related to the integration with Dart DevTools
Projects
None yet
Development

No branches or pull requests

3 participants