Skip to content

Conversation

@sidvishnoi
Copy link
Collaborator

@sidvishnoi sidvishnoi commented Jan 21, 2026

Move specs to root dir, making it easier to discover and update, separate from Astro structure.
Symlink it to public/specification (at build time for platform compatibility), so Astro copies the specifications as is without any processing (including all the includes), so ReSpec documents continue to work.

cc: @lukewarlow @asurkov

Before:

├── public
└── src/pages/specification

After:

├── specification
├── public
│   └── specification -> ../specification
└── src

@netlify
Copy link

netlify bot commented Jan 21, 2026

Deploy Preview for webmonetization-preview ready!

Name Link
🔨 Latest commit c3e7a08
🔍 Latest deploy log https://app.netlify.com/projects/webmonetization-preview/deploys/69788a91ab154e0008904cfd
😎 Deploy Preview https://deploy-preview-633--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.

@sidvishnoi sidvishnoi marked this pull request as ready for review January 21, 2026 13:52
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@JoblersTune Please check this continues to work on Windows. I've tested on MacOS and Linux.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Unfortunately I don't have a Windows machine either.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe one of the tech writers can help, if I remember one of them had a Windows machine, maybe @brad-dow ?

Copy link
Collaborator Author

@sidvishnoi sidvishnoi Jan 22, 2026

Choose a reason for hiding this comment

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

@DarianM Can you check the follow work ok:

  • pnpm start
  • pnpm build; pnpm preview

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, let me do it asap

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

😭 Can you check on /specification/ as well (note: trailing slash)?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Trailing slash has the same behavior

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@brad-dow @DarianM can you please try again?

Copy link
Collaborator

@DarianM DarianM Jan 26, 2026

Choose a reason for hiding this comment

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

/specification preview works fine, there's a /specification 404 dev problem for windows. bun build && bun preview are also fine; using pnpm fine too

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

1d50946 should fix dev as well.

@sidvishnoi sidvishnoi added the specification Work required on specification label Jan 26, 2026
@sidvishnoi sidvishnoi requested a review from brad-dow January 26, 2026 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specification Work required on specification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants