Skip to content

Conversation

@moizhanali
Copy link
Contributor

@moizhanali moizhanali commented Dec 22, 2019

Fixes #2037

Changes: Translated strings.xml into Arabic Language

Screenshot/s for the changes: [Add screenshot/s of the layout where you made changes or a *.gif containing a demonstration]

Checklist: [Please tick following check boxes with [x] if the respective task is completed]

  • I have used resources from strings.xml, dimens.xml and colors.xml without hard-coding them
  • No modifications done at the end of resource files strings.xml, dimens.xml or colors.xml
  • I have reformatted code in every file included in this PR [CTRL+ALT+L]
  • My code does not contain any extra lines or extra spaces
  • I have requested reviews from other members

APK for testing: It doesn't need

Copy link
Member

@nikit19 nikit19 left a comment

Choose a reason for hiding this comment

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

Avoid extra blank lines

@moizhanali
Copy link
Contributor Author

@nikit19 Please review and merge

@moizhanali
Copy link
Contributor Author

@adityastic Please review this :)

@adityastic
Copy link
Contributor

@CloudyPadmal please review

@moizhanali
Copy link
Contributor Author

Just 24 hrs left for the task :( @CloudyPadmal @adityastic @nikit19

@adityastic
Copy link
Contributor

@IzhanAli don't worry I'll extend it for you

@moizhanali
Copy link
Contributor Author

moizhanali commented Dec 28, 2019

@adityastic Thanks

@moizhanali
Copy link
Contributor Author

It's taking longer than expected :(
Can someone please tell how much time will it take to review and approve changes?

@adityastic
Copy link
Contributor

@IzhanAli It's the end of year, people usually arnt working at this time of the year. Please be patient and wait till 5th Jan

@cweitat
Copy link
Contributor

cweitat commented Jan 5, 2020

do you have a screenshot or GIF to show how it looks?

Copy link
Collaborator

@CloudyPadmal CloudyPadmal left a comment

Choose a reason for hiding this comment

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

APKs and Screenshots?

@CloudyPadmal CloudyPadmal added the Status: Review Required Requested reviews from peers and maintainers label Jan 17, 2020
@mariobehling mariobehling merged commit ebf5a79 into fossasia:development Feb 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Review Required Requested reviews from peers and maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Adding language - Arabic

7 participants