Skip to content
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

NTV-235:Ask a question #1419

Merged
merged 6 commits into from Oct 7, 2021
Merged

NTV-235:Ask a question #1419

merged 6 commits into from Oct 7, 2021

Conversation

hadia
Copy link
Contributor

@hadia hadia commented Oct 7, 2021

📲 What

ask the creator a question from the FAQ tab

🛠 How

-Add "SimpleDividerItemDecoration class

  • Add the ask question button in case the user is login
  • The ask question if the project is backed open"MessagesCreatorActivity" else open"MessagesActivity"

👀 See

device-2021-10-07-164545.mp4

📋 QA

  • Open a backed project with FQA in case user login
  • Open a project with FQA in case user login /logout
  • Open a project without FQA in case user login /logout

Story 📖

https://kickstarter.atlassian.net/browse/NTV-235

@codecov
Copy link

codecov bot commented Oct 7, 2021

Codecov Report

Merging #1419 (18fc2f3) into master (f073a66) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1419   +/-   ##
=========================================
  Coverage     72.24%   72.24%           
  Complexity      738      738           
=========================================
  Files           211      211           
  Lines          6085     6085           
  Branches        391      391           
=========================================
  Hits           4396     4396           
  Misses         1551     1551           
  Partials        138      138           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f073a66...18fc2f3. Read the comment docs.

private fun setupRecyclerView() {
binding?.fqaRecyclerView?.adapter = fqaAdapter
requireContext().getDrawable(R.drawable.divider_grey_300_horizontal)?.let {
Copy link
Contributor

Choose a reason for hiding this comment

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

you can substitute
requireContext().getDrawable(R.drawable.divider_grey_300_horizontal)
over
ResourcesCompat.getDrawable(resources, R.drawable.divider_grey_300_horizontal, null)
and remove the SuppressLint rule on line 110

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Contributor

@Arkariang Arkariang left a comment

Choose a reason for hiding this comment

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

Great work @hadia

@hadia hadia merged commit e553c8d into master Oct 7, 2021
@hadia hadia deleted the hadia/NTV-235_Ask_a_question branch October 7, 2021 18:13
Arkariang added a commit that referenced this pull request Oct 7, 2021
…tin/NTV-207-project-overview

* 'master' of github.com:kickstarter/android-oss:
  NTV-235:Ask a question (#1419)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants