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

feat: modify text, font, npm scripts #9

Merged
merged 54 commits into from
Dec 18, 2024
Merged

feat: modify text, font, npm scripts #9

merged 54 commits into from
Dec 18, 2024

Conversation

c2tz
Copy link
Owner

@c2tz c2tz commented Dec 18, 2024

No description provided.

c2tz added 30 commits June 8, 2024 00:10
refactor(SEO): add a restrictive robots.txt

For now i don't want to use SEO for cta.li only when i finish the work.

Reviewed-by: @c2tz
Refs: #2
refactor(PWA): add a manifest.webmanifest file

He shouldn't work well for develop branch (i use staging.cta.li) because start_url is not correctly completed but that not important.

The next PWA scope commit would be the png, webp files.

Reviewed-by: @c2tz
Refs: #2
refactor(PWA): add all manifest.webmanifest requested pics

Second parts with requested pics for manifest.webmanifest files.

Reviewed-by: @c2tz
Refs: #2
test(PWA): rename manifest.w... to manifest.json

Vercel seems to have already a file with that name and that create a conflict so i test with manifest.json for now but that not conventional.

see: https://www.w3.org/TR/appmanifest/#using-a-link-element-to-link-to-a-manifest

Reviewed-by: @c2tz
Refs: #2
feat(pics): add other pics files

Just add other files like favicon apple-touch-icon.png mask.webp (footer), i should make a chore commits for later for remove unused pics.

Reviewed-by: @c2tz
Refs: #2
refactor(SEO): add a sitemap.xml

For now i don't want to use SEO for cta.li only when i finish the work.

Reviewed-by: @c2tz
Refs: #2
test(PWA): try to correct error 401 about manifest

Only test i've found this https://medium.com/@aurelien.delogu/401-error-on-a-webmanifest-file-cb9e3678b9f3 that can be cause by CORS

Reviewed-by: @c2tz
Refs: #2
refactor(SEO): add a sitemap.xml

Just add fonts folder with roboto regular and bold. No need to add mono for now.

That should now be finIsh for #2

Reviewed-by: @c2tz
Refs: #2
ci(prettier): add prettier in ci

Add prettier for workflows just for test now.

Reviewed-by: @c2tz
Refs: #1
test(ci): test prettier

Test prettier now.

Reviewed-by: @c2tz
Refs: #NC
test(ci): change pull request to push

Add prettier for workflows just for test now.

Reviewed-by: @c2tz
Refs: #1
test(ci): change workflow name

Add prettier for workflows just for test now.

Reviewed-by: @c2tz
Refs: #1
test(ci): change permission for action

Add prettier for workflows just for test now.

Reviewed-by: @c2tz
Refs: #1
@c2tz c2tz added the enhancement New feature or request label Dec 18, 2024
Copy link

vercel bot commented Dec 18, 2024

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

Name Status Preview Updated (UTC)
cta-li ✅ Ready (Inspect) Visit Preview Dec 18, 2024 10:28am

@c2tz c2tz merged commit ae7c6b5 into main Dec 18, 2024
1 of 2 checks passed
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.

1 participant