Skip to content

Scrum 130 update polish i 18 n - #46

Merged
Akirus12 merged 14 commits into
masterfrom
SCRUM-130-update-polish-i-18-n
Jun 13, 2026
Merged

Akirus12 merged 14 commits into
masterfrom
SCRUM-130-update-polish-i-18-n

Conversation

@Mateusz-Dobrzynski

Copy link
Copy Markdown
Member

Description

This PR adds missing Polish i18n and internationalizes a number of hard-coded strings.

Testing

- [ ] I have covered code introduced by this pull request with tests.

  • I have run all tests with npm run test && npx playwright test and ensured that they pass consistently.

@Mateusz-Dobrzynski
Mateusz-Dobrzynski force-pushed the SCRUM-130-update-polish-i-18-n branch from 4010d6d to ab49d15 Compare June 10, 2026 21:19
@Mateusz-Dobrzynski

Copy link
Copy Markdown
Member Author

@avocado-rice, please approve and merge my PR before merging yours; this will minimize conflicts to resolve.

@Akirus12 Akirus12 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Everything else looks good, just a small typo to fix

debateId,
}) => {
const [isJudge, setIsJudge] = useState<boolean | null>(null);
const [canSubmitVerdict, setCanSubmitVerdit] = useState<boolean | null>(null);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

setCanSubmitVerdit looks like a typo, change to setCanSubmitVerdict

])
.then(([permissionData, verdictsData]) => {
setIsJudge(Boolean(permissionData));
setCanSubmitVerdit(Boolean(permissionData));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same as above

@Akirus12
Akirus12 merged commit b9f78e2 into master Jun 13, 2026
2 checks passed
@Mateusz-Dobrzynski
Mateusz-Dobrzynski deleted the SCRUM-130-update-polish-i-18-n branch June 14, 2026 02:21
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.

3 participants