Skip to content

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

Merged
merged 1 commit into from
Apr 25, 2019
Merged

Conversation

aggarwalpulkit596
Copy link
Contributor

@aggarwalpulkit596 aggarwalpulkit596 commented Apr 17, 2019

Fixes #1619

Changes: Added Event Faq's in About Event Fragment

Screenshots for the change:
screenshot-1555880744324
screenshot-1555881031765

@auto-label auto-label bot added the feature label Apr 17, 2019
@aggarwalpulkit596 aggarwalpulkit596 force-pushed the faqs branch 3 times, most recently from cd3f591 to ea72ff2 Compare April 19, 2019 03:35
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.

Screenshot? Also avoid unnecessary blank lines in xml

@aggarwalpulkit596
Copy link
Contributor Author

This api is not yet merged to prod server so waiting for that otherwise it is complete

@iamareebjamal
Copy link
Member

You don't use prod API, you use dev API

@aggarwalpulkit596
Copy link
Contributor Author

I'm using prod server for testing

@aggarwalpulkit596
Copy link
Contributor Author

@nikit19 review now

@iamareebjamal
Copy link
Member

I'm using prod server for testing

You shouldn't

@aggarwalpulkit596
Copy link
Contributor Author

@iamareebjamal okay I'll keep that in mind for future PR's

@aggarwalpulkit596
Copy link
Contributor Author

@iamareebjamal review

@iamareebjamal
Copy link
Member

There should be an icon in the action bar which should open the FAQ fragment

@aggarwalpulkit596
Copy link
Contributor Author

An entirely new fragment for faq ? Currently I'm showing them about event fargment

@iamareebjamal
Copy link
Member

Yeah, new fragment

@aggarwalpulkit596
Copy link
Contributor Author

@nikit19 @iamareebjamal updated

@aggarwalpulkit596 aggarwalpulkit596 force-pushed the faqs branch 2 times, most recently from bbe8a3d to dd61fec Compare April 21, 2019 21:18
@iamareebjamal
Copy link
Member

Text in action bar is fine. Remove the text in Fragment

@aggarwalpulkit596
Copy link
Contributor Author

@liveHarshit review

@aggarwalpulkit596
Copy link
Contributor Author

@iamareebjamal updated

@aggarwalpulkit596
Copy link
Contributor Author

@iamareebjamal anything left out?

@iamareebjamal iamareebjamal changed the title feat: Add Event FAQ's feat: Add Event FAQs Apr 25, 2019
@iamareebjamal iamareebjamal merged commit 6d1b5fb into fossasia:development Apr 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add FAQ Section
3 participants