-
Notifications
You must be signed in to change notification settings - Fork 553
feat: Add Event FAQs #1623
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
feat: Add Event FAQs #1623
Conversation
cd3f591
to
ea72ff2
Compare
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.
Screenshot? Also avoid unnecessary blank lines in xml
This api is not yet merged to prod server so waiting for that otherwise it is complete |
You don't use prod API, you use dev API |
I'm using prod server for testing |
@nikit19 review now |
You shouldn't |
@iamareebjamal okay I'll keep that in mind for future PR's |
@iamareebjamal review |
There should be an icon in the action bar which should open the FAQ fragment |
An entirely new fragment for faq ? Currently I'm showing them about event fargment |
Yeah, new fragment |
@nikit19 @iamareebjamal updated |
bbe8a3d
to
dd61fec
Compare
Text in action bar is fine. Remove the text in Fragment |
@liveHarshit review |
@iamareebjamal updated |
@iamareebjamal anything left out? |
Fixes #1619
Changes: Added Event Faq's in About Event Fragment
Screenshots for the change:

