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

incorporation localization of texts using of the library i18n #292

Merged
merged 7 commits into from
Jan 18, 2023

Conversation

Viterbo
Copy link
Collaborator

@Viterbo Viterbo commented Jan 10, 2023

Fixes #91

Description

Telos scan needs to include internationalization and for that i18n was incorporated and all hard-coded texts were changed for a call to the $t() function with the corresponding path.

Requirements

This PR needs to be merged after merging the incorporation of the library i18n PR

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have cleaned up the code in the areas my change touches
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings
  • I have removed any unnecessary console messages

@Viterbo Viterbo self-assigned this Jan 10, 2023
@Viterbo Viterbo linked an issue Jan 10, 2023 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for teloscan-stage ready!

Name Link
🔨 Latest commit 9c2ec6d
🔍 Latest deploy log https://app.netlify.com/sites/teloscan-stage/deploys/63c5c8b8ba682e00096fed00
😎 Deploy Preview https://deploy-preview-292--teloscan-stage.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@netlify
Copy link

netlify bot commented Jan 10, 2023

Deploy Preview for dev-mainnet-teloscan ready!

Name Link
🔨 Latest commit 9c2ec6d
🔍 Latest deploy log https://app.netlify.com/sites/dev-mainnet-teloscan/deploys/63c5c8b8934cc10009faf53b
😎 Deploy Preview https://deploy-preview-292--dev-mainnet-teloscan.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 11, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9c2ec6d
Status: ✅  Deploy successful!
Preview URL: https://da202db5.teloscan.pages.dev
Branch Preview URL: https://91-add-i18n-localizations.teloscan.pages.dev

View logs

@Viterbo Viterbo changed the title incorporation of the library i18n and some translations (not ready yet) incorporation localization of texts using of the library i18n Jan 11, 2023
@Viterbo Viterbo marked this pull request as ready for review January 11, 2023 02:00
@donnyquixotic
Copy link
Contributor

donnyquixotic commented Jan 11, 2023

@Viterbo check syntax in templates, text fields aren't rendering. examples:
image
image

Copy link
Contributor

@donnyquixotic donnyquixotic left a comment

Choose a reason for hiding this comment

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

See comment

Copy link
Contributor

@karynemayer karynemayer left a comment

Choose a reason for hiding this comment

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

Please review the lines listed (I couldn't put a comment in the lines that weren't active on the diff :( ). I also mentioned console.error messages that might need to be showed in the UI and thus, translated.

package.json Outdated Show resolved Hide resolved
src/boot/i18n.js Outdated Show resolved Hide resolved
src/boot/i18n.js Show resolved Hide resolved
src/components/ConfirmationDialog.vue Show resolved Hide resolved
src/components/ConnectButton.vue Show resolved Hide resolved
src/pages/staking/UnstakeForm.vue Show resolved Hide resolved
src/pages/staking/UnstakeForm.vue Outdated Show resolved Hide resolved
src/pages/staking/UnstakeForm.vue Show resolved Hide resolved
src/pages/staking/UnstakeForm.vue Show resolved Hide resolved
src/pages/staking/WithdrawPage.vue Show resolved Hide resolved
@Viterbo Viterbo marked this pull request as draft January 12, 2023 22:09
@Viterbo Viterbo marked this pull request as ready for review January 12, 2023 22:18
donnyquixotic
donnyquixotic previously approved these changes Jan 13, 2023
Copy link
Contributor

@ezra-sg ezra-sg left a comment

Choose a reason for hiding this comment

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

this is a lot of work, nicely done. I also appreciate that you added notifications for error messages :)
i have some initial feedback, and also I am having issues with the branch in general (we will talk about that over telegram)

src/components/inputs/BooleanArrayInput.vue Outdated Show resolved Hide resolved
src/pages/AccountAddress.vue Outdated Show resolved Hide resolved
src/pages/AccountAddress.vue Outdated Show resolved Hide resolved
src/pages/staking/StakeForm.vue Outdated Show resolved Hide resolved
src/pages/staking/StakeForm.vue Outdated Show resolved Hide resolved
@Viterbo Viterbo requested a review from ezra-sg January 13, 2023 22:41
donnyquixotic
donnyquixotic previously approved these changes Jan 13, 2023
@ezra-sg
Copy link
Contributor

ezra-sg commented Jan 16, 2023

contract tab is missing some localizations on route /address/0xf42C4b1D1F93e9e31208Ae6907546fF0F675205b#contract:
image

also the tabs there seem to be broken

@ezra-sg
Copy link
Contributor

ezra-sg commented Jan 16, 2023

tx details page, route /tx/0x59feb330964cc8d3922cbb22207dc0070b1d744bf865ba6726dba79d331cfd29#general

image

as well as Details tab
image

@donnyquixotic donnyquixotic self-requested a review January 17, 2023 16:21
Copy link
Contributor

@ezra-sg ezra-sg left a comment

Choose a reason for hiding this comment

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

very nice work

@karynemayer karynemayer merged commit 5434216 into dev Jan 18, 2023
@karynemayer karynemayer deleted the 91-add-i18n-localizations branch January 18, 2023 02:20
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.

Add i18n / localizations
4 participants