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

ref(flags): Simplify onboarding and drawer platform lists #88756

Merged
merged 1 commit into from
Apr 7, 2025

Conversation

aliu39
Copy link
Member

@aliu39 aliu39 commented Apr 3, 2025

Simplifies these lists by filtering static/app/data/platforms by javascript and python prefixes. Note our PlatformKeys use the node- prefix for backend JS, which we don't support yet.

Removes 'other' from drawer platforms. We don't have a CTA for other, so there's no point always showing the empty state.

With these changes we're also including these python platforms we didn't have before:
asgi awslambda celery chalice gcpfunctions pylons pymongo rq serverless tryton wsgi

@aliu39 aliu39 requested a review from a team as a code owner April 3, 2025 22:28
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 3, 2025
@aliu39 aliu39 requested a review from a team April 3, 2025 22:29
Copy link
Member

@michellewzhang michellewzhang left a comment

Choose a reason for hiding this comment

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

do we need both if they're the same arrays?

@aliu39 aliu39 requested review from bruno-garcia, a team and cmanallen April 3, 2025 22:31
@aliu39
Copy link
Member Author

aliu39 commented Apr 3, 2025

do we need both if they're the same arrays?

I think it's good to have them separate in case we support a new platform, but haven't made gettingStartedDocs for it yet.

@aliu39
Copy link
Member Author

aliu39 commented Apr 3, 2025

With these changes we're also including these python platforms we didn't have before:
asgi awslambda celery chalice gcpfunctions pylons pymongo rq serverless tryton wsgi

@cmanallen @bruno-garcia any reason these were excluded before? We include all python in the docs site

@cmanallen
Copy link
Member

@aliu39 Probably fine to include them.

@aliu39 aliu39 merged commit c96d159 into master Apr 7, 2025
42 checks passed
@aliu39 aliu39 deleted the aliu/flag-platforms branch April 7, 2025 18:47
andrewshie-sentry pushed a commit that referenced this pull request Apr 8, 2025
Simplifies these lists by filtering `static/app/data/platforms` by
javascript and python prefixes. Note our PlatformKeys use the `node-`
prefix for backend JS, which we don't support yet.

Removes 'other' from drawer platforms. We don't have a CTA for other, so
there's no point always showing the empty state.

With these changes we're also including these python platforms we didn't
have before:
`asgi awslambda celery chalice gcpfunctions pylons pymongo rq serverless
tryton wsgi`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants