Skip to content

revert: new tools page that loads WM tools#533

Merged
DarianM merged 4 commits intoWICG:mainfrom
DarianM:tools-page
Jun 12, 2025
Merged

revert: new tools page that loads WM tools#533
DarianM merged 4 commits intoWICG:mainfrom
DarianM:tools-page

Conversation

@DarianM
Copy link
Collaborator

@DarianM DarianM commented May 29, 2025

Description

This reverts PR #504 ,/tools path no longer needed as astro page.
Instead, add a notification message for the Contributors telling them about the tools repository.

@netlify
Copy link

netlify bot commented May 29, 2025

Deploy Preview for webmonetization-preview ready!

Name Link
🔨 Latest commit ed2b515
🔍 Latest deploy log https://app.netlify.com/projects/webmonetization-preview/deploys/684aa5a1a7adac0008c5b348
😎 Deploy Preview https://deploy-preview-533--webmonetization-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@DarianM DarianM requested review from JoblersTune and sidvishnoi May 29, 2025 11:34
@sidvishnoi
Copy link
Collaborator

Can we instead of deleting, turn this to a simple page with a message: pages under /tools are managed by tools repo, or something? This way, contributors will have some guidance if they try to edit /tools here.

@JoblersTune
Copy link
Collaborator

Either we go with @sidvishnoi's suggestion or we can at very least update the README to include that information.

@DarianM DarianM merged commit b6407ad into WICG:main Jun 12, 2025
5 of 6 checks passed
@DarianM DarianM deleted the tools-page branch June 12, 2025 21:04
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.

3 participants