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

refactor: temporarily remove Web Editor platform from supportedPlatformsData #51

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

charlottewiltshire0
Copy link
Collaborator

PR Content:

@charlottewiltshire0 charlottewiltshire0 added the enhancement New feature or request label Jan 14, 2025
@charlottewiltshire0 charlottewiltshire0 marked this pull request as ready for review January 14, 2025 16:03
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Temporarily removes Web Editor platform from supported platforms list while maintaining configuration for future re-enablement.

  • Commented out Web Editor section in /constants/supportedPlatformsData.ts with clear documentation for future restoration
  • Removed unused getPlatformDownloadLink import to maintain code cleanliness
  • Platform entries for Android, Windows, macOS, and Linux remain unchanged

💡 (2/5) Greptile learns from your feedback when you react with 👍/👎!

1 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile

// Uncomment this section when Web Editor support is added.
// {
// label: "Web Editor",
// href: getPlatformDownloadLink("webEditor"),
Copy link

Choose a reason for hiding this comment

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

logic: getPlatformDownloadLink import was removed but still referenced in commented code. Consider removing the function call or keeping the import commented

Copy link

Deploying redot-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8ef3b84
Status: ✅  Deploy successful!
Preview URL: https://9e2bd2c1.redot-website.pages.dev
Branch Preview URL: https://chore-web-editor-hide.redot-website.pages.dev

View logs

@charlottewiltshire0 charlottewiltshire0 merged commit 5730acb into master Jan 14, 2025
5 checks passed
@charlottewiltshire0 charlottewiltshire0 deleted the chore/web-editor-hide branch January 14, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants