Skip to content

Update and complete README-PL #1031

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

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

Conversation

kasiawalsh
Copy link

Checklist:
I have noticed that the Polish translation is incomplete. I would like to complete the content and refine the existing translation.

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the main branch of freeCodeCamp.

Closes #1029

@kasiawalsh kasiawalsh marked this pull request as draft June 6, 2025 19:56
README-PL.md Outdated

**Zawartość**
**Spis treści**
Copy link
Contributor

Choose a reason for hiding this comment

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

isn't the content below just links? don't really see how it's a spis treści

Copy link
Author

Choose a reason for hiding this comment

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

No worries, it's still WIP, original Zawartość is good, I can revert
Let me know if you disagree with anything else, happy to change back, a lot of it preferential

Copy link
Author

Choose a reason for hiding this comment

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

Hi, could you please review and let me know if you don't agree with anything?
I've added some content based on the English source file.
Thanks!

Copy link
Contributor

Choose a reason for hiding this comment

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

sure, give me a bit

Copy link
Author

Choose a reason for hiding this comment

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

Nice, thank you for the review, well spotted errors :) It's always good to have a second pair of eyes.
This is my first time contributing to open source

Copy link
Author

Choose a reason for hiding this comment

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

Are you happy to approve the PR?

Copy link
Contributor

Choose a reason for hiding this comment

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

i don't have write access 😆

Copy link
Author

Choose a reason for hiding this comment

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

Oh ok, I have commented for maintainers to review

@kasiawalsh kasiawalsh marked this pull request as ready for review June 10, 2025 18:38
@kasiawalsh kasiawalsh requested a review from table106 June 10, 2025 18:38
Copy link
Contributor

@table106 table106 left a comment

Choose a reason for hiding this comment

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

please address the issues i mentioned

@table106
Copy link
Contributor

also let me know if i can improve my reviewing, first time doing this

Copy link
Contributor

@table106 table106 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@kasiawalsh
Copy link
Author

kasiawalsh commented Jun 11, 2025

Hello @maintainers, would you be able to review this PR? Is there anything else needed to merge? Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improving and completing README-PL.md
2 participants