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

docs: add <meta> tags (Group 2) #951

Merged
merged 4 commits into from
Mar 20, 2024

Conversation

marcpicaud
Copy link
Contributor

Hey there!

Summary

This PR adds <meta> tags to the pages listed as "Group 2" in #791 .

I kept the descriptions under 150 chars to avoid them being truncated, and I tried to keep them as descriptive, catchy and SEO friendly as possible. Feedback welcome if something can get improved !

Location

For the pages listed as "Group 2" in flybywiresim#791
Copy link

vercel bot commented Mar 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 17, 2024 6:08pm

Copy link
Member

@Valastiri Valastiri left a comment

Choose a reason for hiding this comment

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

some suggestions and branding of our flyPadOS 3 changes

docs/fbw-a32nx/feature-guides/cFMS.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flight-planning.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/charts.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/dashboard.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/dispatch.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/ground.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/online-atc.md Outdated Show resolved Hide resolved
docs/fbw-a32nx/feature-guides/flypados3/performance.md Outdated Show resolved Hide resolved
@marcpicaud
Copy link
Contributor Author

Hi there,

Appreciate the time you took to review the texts, thank you!
I've pushed the suggestions just now, sorry for the delay :)

Cheers

@marcpicaud marcpicaud changed the title docs: add <meta> tags (group 2) docs: add <meta> tags (Group 2) Mar 17, 2024
@Valastiri
Copy link
Member

No worries and lgtm! Appreciate it.

@github-actions github-actions bot removed the Review Required PR Check Label label Mar 20, 2024
@github-actions github-actions bot merged commit 5a2ed90 into flybywiresim:primary Mar 20, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants