Skip to content

feat: add translation for Brazilian Portuguese (pt-BR) #182

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.

Sign up for GitHub

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

Closed
tardisblink wants to merge 5 commits into from
Closed

feat: add translation for Brazilian Portuguese (pt-BR) #182

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
aae8662
feat: add translation for Brazilian Portuguese (pt-BR)
tardisblink Dec 14, 2024
7c81426
feat: add translation for Brazilian Portuguese (pt-BR)
tardisblink Dec 14, 2024
e48c35d
Merge branch 'develop' of https://github.com/tardisblink/shield-oauth…
tardisblink Dec 14, 2024
86a30b2
Merge branch 'develop' of https://github.com/tardisblink/shield-oauth…
tardisblink Dec 14, 2024
8a66a96
Merge branch 'develop' of https://github.com/tardisblink/shield-oauth…
tardisblink Dec 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 44 additions & 0 deletions src/Language/pt-BR/ShieldOAuthLang.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
<?php

declare(strict_types=1);

/**
* This file is part of Shield OAuth.
*
* (c) Datamweb <[email protected]>
*
* For the full copyright and license information, please view
* the LICENSE file that was distributed with this source code.
*/

// ShieldOAuthLang language file
return [
// Errors List
'unknown' => 'Erro desconhecido!',
'Callback' => [
'oauth_class_not_set' => 'Ocorreu um erro, parece que a classe OAuth não está configurada.',
'anti_forgery' => 'Sua solicitação foi detectada como falsa. Pedimos desculpas!',
'account_not_found' => 'Não há nenhuma conta registrada com o e-mail "{0}".',
'access_denied' => 'Autenticação cancelada! Você recusou as permissões de {0}.',
],

// ShieldOAuthButton in views
'other' => 'Outro',
'login_by' => 'Entrar com: ',
'register_by' => 'Registrar com: ',

// Errors List For all OAuth
'Github' => [
'github' => 'GitHub',
'not_allow' => 'Agora você não pode entrar ou se registrar com o GitHub!',
],
'Google' => [
'google' => 'Google',
'not_allow' => 'Agora você não pode entrar ou se registrar com o Google!',
],
// 'Yahoo' => [
// 'yahoo' => 'Yahoo',
// 'not_allow' => 'Agora você não pode entrar ou se registrar com o Yahoo!',
// ],
Comment on lines +30 to +42
Copy link

@coderabbitai coderabbitai bot Dec 14, 2024

Choose a reason for hiding this comment

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

💡 Codebase verification

Remove commented Yahoo OAuth sections

The Yahoo OAuth provider is consistently commented out across all language files and the configuration file, indicating it's not an active or planned feature. The commented sections should be removed to maintain clean code:

  • Yahoo configuration is commented out in src/Config/ShieldOAuthConfig.php
  • Yahoo translations are commented out in all language files:
    • src/Language/en/ShieldOAuthLang.php
    • src/Language/fa/ShieldOAuthLang.php
    • src/Language/fr/ShieldOAuthLang.php
    • src/Language/id/ShieldOAuthLang.php
    • src/Language/pt-BR/ShieldOAuthLang.php
🔗 Analysis chain

Query about commented Yahoo section.

The translations for GitHub and Google are well-implemented. However, there's a commented-out Yahoo section that might need clarification:

  • Is Yahoo support planned for the future?
  • Should this commented section be removed if Yahoo support isn't imminent?

Let's check the content of these files to understand the Yahoo implementation status better.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Check Yahoo implementation status across the project
# Expected: Find if Yahoo OAuth is implemented or planned

# Search for Yahoo-related implementations
rg -l "Yahoo" --type php

Length of output: 240

Script:

#!/bin/bash
# Check the content of ShieldOAuthConfig.php for Yahoo configuration
echo "=== ShieldOAuthConfig.php ==="
rg -A 5 -B 5 "Yahoo" src/Config/ShieldOAuthConfig.php

# Check if Yahoo is commented out in other language files
echo -e "\n=== Language files Yahoo sections ==="
for file in src/Language/*/ShieldOAuthLang.php; do
    echo -e "\n--- $file ---"
    rg -A 3 -B 1 "Yahoo" "$file"
done

Length of output: 2397

];

Loading