-
Notifications
You must be signed in to change notification settings - Fork 429
fixed hardcoded text lint error #1102 #1321
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please go through contribution guidelines and submit a pull request only after you are assigned to an issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hsymphonyk Travis build is failing. Please fix it.
Also, squash the commits.
2c3f9cc
to
172df7a
Compare
Hello, @Rimjhim28 , I have done the changes requested. Please review this.Thank you. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have added some comments. Be sure that you squash the commits at the end.
172df7a
to
d573726
Compare
@Rimjhim28 I have squashed the commits. Please review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good Job @hsymphonyk! 🎉
Description
Changed the hardcoded texts using resources.
Fixes #1102
Type of Change:
How Has This Been Tested?
Application was built successfully in android studio without any errors.
Checklist:
Code/Quality Assurance Only