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

[pt-br] update of web/http/cookies #22449

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[pt-br] update of web/http/cookies #22449

wants to merge 4 commits into from

Conversation

vittxr
Copy link

@vittxr vittxr commented Jul 17, 2024

  • Improve Lax attribute description: I improved it based on this Stack Overflow answer.
  • Fix note card with the backend languages: it is equal to English docs.
  • Fix SameSite description (in the English docs, Lax is the default value of SameSite).

- Improve `Lax` attribute description.
- Fix `note` card with the backend languages.
- Fix `SameSite` description
@vittxr vittxr requested a review from a team as a code owner July 17, 2024 00:13
@vittxr vittxr requested review from josielrocha and removed request for a team July 17, 2024 00:13
@github-actions github-actions bot added the l10n-pt-br Issues related to Brazilian Portuguese label Jul 17, 2024
Copy link
Contributor

github-actions bot commented Jul 17, 2024

Preview URLs

Flaws (19)

URL: /pt-BR/docs/Web/HTTP/Cookies
Title: Cookies HTTP
Flaw count: 19

  • macros:
    • /pt-BR/docs/Web/HTTP/Basics_of_HTTP/Data_URLs does not exist but fell back to /en-US/docs/Web/HTTP/Basics_of_HTTP/Data_URLs
    • /pt-BR/docs/Web/HTTP/Basics_of_HTTP/Choosing_between_www_and_non-www_URLs does not exist but fell back to /en-US/docs/Web/HTTP/Basics_of_HTTP/Choosing_between_www_and_non-www_URLs
    • /pt-BR/docs/Web/HTTP/Basics_of_HTTP/Evolution_of_HTTP does not exist but fell back to /en-US/docs/Web/HTTP/Basics_of_HTTP/Evolution_of_HTTP
    • /pt-BR/docs/Web/HTTP/Protocol_upgrade_mechanism does not exist but fell back to /en-US/docs/Web/HTTP/Protocol_upgrade_mechanism
    • /pt-BR/docs/Web/HTTP/Conditional_requests does not exist but fell back to /en-US/docs/Web/HTTP/Conditional_requests
    • and 11 more flaws omitted
  • broken_links:
    • Can't resolve /pt-BR/docs/Tools/Storage_Inspector
    • No need for the pathname in anchor links if it's the same page
    • Can't resolve /pt-BR/docs/Tools/Storage_Inspector
External URLs (4)

URL: /pt-BR/docs/Web/HTTP/Cookies
Title: Cookies HTTP

(comment last updated: 2024-08-06 12:45:53)

Copy link
Collaborator

@josielrocha josielrocha left a comment

Choose a reason for hiding this comment

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

Hey @vittxr!

Thank you for your help!

We are changing the way we write note blocks. Can you please check the suggestion I left in this PR?

files/pt-br/web/http/cookies/index.md Outdated Show resolved Hide resolved
update note block

Co-authored-by: Josiel Rocha <[email protected]>
@vittxr
Copy link
Author

vittxr commented Jul 29, 2024

@josielrocha No problem! I'm glad to help :)

I approved the suggestions

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 8, 2024
Copy link
Contributor

github-actions bot commented Aug 8, 2024

This pull request has merge conflicts that must be resolved before it can be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-pt-br Issues related to Brazilian Portuguese merge conflicts 🚧 This pull request has merge conflicts that must be resolved.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants