Skip to content

Conversation

@suisin-deriv
Copy link
Contributor

Changes:

Add German Language to app.deriv

Screenshots:

Please provide some screenshots of the change.

@vercel
Copy link

vercel bot commented Aug 17, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Sep 22, 2023 8:35am

@github-actions
Copy link
Contributor

github-actions bot commented Aug 17, 2023

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 17
🟧 Accessibility 75
🟧 Best practices 83
🟧 SEO 85
🟢 PWA 90

Lighthouse ran with https://deriv-app-git-fork-suisin-deriv-suisin-wall1529addgerman-8592d2.binary.sx/

@coveralls
Copy link

coveralls commented Aug 17, 2023

Coverage Status

coverage: 10.233% (-0.003%) from 10.236% when pulling fa20d24 on suisin-deriv:suisin/WALL1529/add_german_language into cd3154a on binary-com:master.

amina-deriv
amina-deriv previously approved these changes Aug 18, 2023
Copy link
Contributor

@matin-deriv matin-deriv left a comment

Choose a reason for hiding this comment

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

Pending from this PR:

  • Add this new language to P2P since they have a separate translation procedure.
  • Wait for SmartCharts release and include the new version of that repo to this PR

Other than that, LGTM

maryia-deriv
maryia-deriv previously approved these changes Aug 21, 2023
farrah-deriv
farrah-deriv previously approved these changes Aug 22, 2023
maryia-deriv
maryia-deriv previously approved these changes Sep 11, 2023
farrah-deriv
farrah-deriv previously approved these changes Sep 11, 2023
maryia-deriv
maryia-deriv previously approved these changes Sep 12, 2023
amina-deriv
amina-deriv previously approved these changes Sep 12, 2023
matin-deriv
matin-deriv previously approved these changes Sep 12, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
3.7% 3.7% Duplication

@ali-hosseini-deriv ali-hosseini-deriv merged commit 095dc92 into deriv-com:master Sep 22, 2023
vinu-deriv pushed a commit that referenced this pull request Oct 10, 2023
* chore: add german language for deriv.app

* chore: empty

* chore: remove ? because it is fixed in master

* chore: add German Language in p2p/i18 package as well

* chore: update charts to latest version

* chore: resolve bug from Germany language

* chore: update scss to remove padding in text

* chore: package-lock.json

* chore: resolve text overlapping screen in mobile

* chore: update scss to fix title issue

* chore: edit scss for text

* chore: update charts version

* chore: remove text-align center

* chore: update charts package

* chore: update scss and Text component
vinu-deriv pushed a commit that referenced this pull request Oct 10, 2023
* chore: add german language for deriv.app

* chore: empty

* chore: remove ? because it is fixed in master

* chore: add German Language in p2p/i18 package as well

* chore: update charts to latest version

* chore: resolve bug from Germany language

* chore: update scss to remove padding in text

* chore: package-lock.json

* chore: resolve text overlapping screen in mobile

* chore: update scss to fix title issue

* chore: edit scss for text

* chore: update charts version

* chore: remove text-align center

* chore: update charts package

* chore: update scss and Text component
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants