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

v2 migration: guides/building to next #1731

Draft
wants to merge 45 commits into
base: v2
Choose a base branch
from

Conversation

vasfvitor
Copy link
Contributor

@vasfvitor vasfvitor commented Dec 5, 2023

There's no need to review now, just need a feedback on the last item on TODO list (Revise Copy)

What kind of changes does this PR include?

  • New or updated content
  • Changes to the docs site code

Description

Diff at: f101f88
The Checklist' TOC changes on app-size.md was made automagically so I decided to keep them if that's ok

Pages:
https://tauri.app/v1/guides/building/
https://tauri.app/v1/guides/building/windows
https://tauri.app/v1/guides/building/macos
https://tauri.app/v1/guides/building/linux
https://tauri.app/v1/guides/building/cross-platform
https://tauri.app/v1/guides/building/app-size
https://tauri.app/v1/guides/building/sidecar
https://tauri.app/v1/guides/building/resources

TODO:

  • Redirects
  • Update/Improve index page (card positions)
  • Fix links and navigation
  • i18n pages, if any
  • Revise changes
  • Revise Copy? - mostly because it mentions Allowlist and this has changed, plus, there may be other changes.

Now I'm wondering, if we change those, how will it be for folks utilizing v1?

I was thinking of transforming _tauri-build.mdx into a fragment as cta.mdx but leave only tabs, then text contents will be placed on each two page with I guess it's better to keep it as is just to avoid text duplication and possible different translations

Copy link

netlify bot commented Dec 5, 2023

Deploy Preview for tauri-docs-starlight ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 17d1a89
🔍 Latest deploy log https://app.netlify.com/sites/tauri-docs-starlight/deploys/65e89af907424300082cbef5
😎 Deploy Preview https://deploy-preview-1731--tauri-docs-starlight.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 100 (no change from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@github-actions github-actions bot added the i18n Is this related to translations? label Dec 11, 2023
@github-actions github-actions bot removed the i18n Is this related to translations? label Dec 11, 2023
Comment on lines 273 to 275
{/* TODO: Check distribute links once those are in */}
[windows code signing with github actions]: /guides/distribute/sign-windows#bonus-sign-your-application-with-github-actions
[macos code signing with github actions]: /guides/distribute/sign-macos#example
Copy link
Contributor Author

@vasfvitor vasfvitor Dec 20, 2023

Choose a reason for hiding this comment

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

links breaking the build, from /guides/distrubute to be migrated

Copy link
Member

Choose a reason for hiding this comment

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

Let's point to prod for now (https://tauri.app/foo...) to get this through and then we can update later.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so, I was looking at the pages that was changed over and turns out if I understand correctly from the Discord thread, it's the Code Signing pages. Now we can link them here in /build

@vasfvitor vasfvitor changed the title Draft: v2 migration: guides/building to next v2 migration: guides/building to next Dec 24, 2023
@vasfvitor vasfvitor marked this pull request as ready for review December 24, 2023 04:57
@lorenzolewis
Copy link
Member

@vasfvitor is this one ready for a full review now?

@vasfvitor
Copy link
Contributor Author

@lorenzolewis yeah it is

@simonhyll simonhyll added this to the Version 2.0 milestone Feb 26, 2024
@simonhyll simonhyll requested a review from a team February 29, 2024 00:59
@vasfvitor
Copy link
Contributor Author

consider #1961 changes

@vasfvitor vasfvitor marked this pull request as draft May 23, 2024 14:53
@simonhyll simonhyll added enhancement Does it add or improve content? and removed i18n Is this related to translations? labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Does it add or improve content?
Projects
Status: 🏗️ In progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants