Skip to content

Add Finnish translations for contributions and streaks#888

Open
akkinyu2002 wants to merge 2 commits intoDenverCoder1:mainfrom
akkinyu2002:patch-2
Open

Add Finnish translations for contributions and streaks#888
akkinyu2002 wants to merge 2 commits intoDenverCoder1:mainfrom
akkinyu2002:patch-2

Conversation

@akkinyu2002
Copy link

@akkinyu2002 akkinyu2002 commented Mar 8, 2026 •
edited
Loading

Its finnish

Description

Fixes #

Type of change

  • Bug fix (added a non-breaking change which fixes an issue)
  • New feature (added a non-breaking change which adds functionality)
  • Updated documentation (updated the readme, templates, or other repo files)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

  • Tested locally with a valid username
  • Tested locally with an invalid username
  • Ran tests with composer test
  • Added or updated test cases to test new features

Checklist:

  • I have checked to make sure no other pull requests are open for this issue
  • The code is properly formatted and is consistent with the existing code style
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

Screenshots

Copilot AI review requested due to automatic review settings March 8, 2026 04:16
@akkinyu2002
Copy link
Author

akkinyu2002 commented Mar 8, 2026

I just added finnish lang

Copilot AI reviewed Mar 8, 2026
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds Finnish (fi) locale strings to the translations map used by the streak stats card, enabling Finnish labels for contribution and streak metrics.

Changes:

  • Added a new fi translation block for contribution/streak-related labels in src/translations.php.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

src/translations.php Outdated
Comment on lines +172 to +175
"Longest Week Streak" => "Pisin viikon sarja",
"Present" => "Nykyisin",
"Excluding {days}" => "Pois lukien {days}",
],
Copy link

Copilot AI Mar 8, 2026

Choose a reason for hiding this comment

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

The closing bracket for the fi locale array is not indented consistently with other locale blocks (it should align with the opening "fi" => [ line). Please adjust indentation to match surrounding locale blocks.

Copilot uses AI. Check for mistakes.
Copy link
Author

@akkinyu2002 akkinyu2002 Mar 8, 2026

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

Copy link
Author

@akkinyu2002 akkinyu2002 Mar 8, 2026

Choose a reason for hiding this comment

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

@copilot open a new pull request to apply changes based on this feedback

@akkinyu2002
Copy link
Author

akkinyu2002 commented Mar 8, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

@akkinyu2002
Update src/translations.php
fb06aeb 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@akkinyu2002
Copy link
Author

akkinyu2002 commented Mar 8, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@akkinyu2002