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-397: Clickable images on campaign #1541

Merged
merged 1 commit into from
Feb 11, 2022

Conversation

Arkariang
Copy link
Contributor

@Arkariang Arkariang commented Feb 10, 2022

πŸ“² What

  • The image can be clickable if there is an url

πŸ‘€ See

clickableImage.mp4

| | |

πŸ“‹ QA

Check the links to Discord, Twitter, and Steam

Story πŸ“–

NTV-397

@codecov
Copy link

codecov bot commented Feb 10, 2022

Codecov Report

Merging #1541 (95ec326) into master (b90e63d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1541   +/-   ##
=========================================
  Coverage     68.93%   68.93%           
  Complexity      442      442           
=========================================
  Files           169      169           
  Lines          4301     4301           
  Branches        231      231           
=========================================
  Hits           2965     2965           
  Misses         1271     1271           
  Partials         65       65           

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 b90e63d...95ec326. Read the comment docs.

@Arkariang Arkariang marked this pull request as ready for review February 11, 2022 17:27
@Arkariang Arkariang merged commit 2a47258 into master Feb 11, 2022
@Arkariang Arkariang deleted the imartin/NTV-397-clicable-image branch February 11, 2022 18:10
Arkariang added a commit that referenced this pull request Mar 3, 2022
…tin/NTV-319-Substitute-BoolSharedPrefs-BoolDataStore

* 'master' of github.com:kickstarter/android-oss: (60 commits)
  NTV-366: DiscoveryParams migrated to kotlin and deprecated autoparcel (#1554)
  NTV-383: Migrate Message.java to kotlin and deprecate Autoparcel (#1551)
  NTV-415 :Jacoco code coverage only looking throw Java classes (#1550)
  NTV-399  Reply comments created with invalid commentable (#1549)
  NTV-414: /v1/activities was called from the app several times #1548
  NTV-412: Hide old campaign links #1547
  NTV-364: Discovery Activity fetchCategories from GraphQL (#1546)
  NTV-410: Upgrade stripe SDK (#1544)
  NTV-406: Remove old ProjectActivity and clean feature flag (#1543)
  NTV-395: UI – Out of place image captions (#1542)
  NTV-397: Clickable images on campaign (#1541)
  NTV-350 :Add Save Project deep link (#1540)
  NTV-360: Zoom on campaign images (#1539)
  Fix friend avatar issue (#1538)
  NTV-349  Add Save Project button to cards on Discovery (#1537)
  NTV-372: Update braze/segment dependency to 10.0.0 (#1532)
  NTV-188: Project story (#1536)
  added percentage funded (#1535)
  NTV-365  Migrate to Kotlin `DiscoveryAdapter.java` (#1531)
  NTV-367: Discovery drawer adapter (#1530)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants