eigen icon indicating copy to clipboard operation
eigen copied to clipboard

feat: news in app [DO NOT MERGE]

Open brainbicycle opened this issue 1 year ago • 1 comments

This PR resolves []

Description

Adds an news component to the home feed below the fold above editorial and a news screen that shows all news articles.

https://github.com/artsy/eigen/assets/49686530/9d6a704f-45d2-4ad6-9e70-8935e203f06e

Follow-ups (if we want to ship):

  • Fix tracking copied over from editorial
  • Tests
  • Feature flag
  • Review with design and product if this is something we want and what we want changed

PR Checklist

  • [ ] I have tested my changes on iOS and Android.
  • [ ] I hid my changes behind a feature flag, or they don't need one.
  • [x] I have included screenshots or videos, or I have not changed the UI.
  • [ ] I have added tests, or my changes don't require any.
  • [x] I added an app state migration, or my changes do not require one.
  • [x] I have documented any follow-up work that this PR will require, or it does not require any.
  • [x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

  • [ ] I would like at least one of the reviewers to run this PR on the simulator or device.
Changelog updates

Changelog updates

Cross-platform user-facing changes

  • Home feed news module and news screen - Brian

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

brainbicycle avatar Jan 18 '24 09:01 brainbicycle

This PR contains the following changes:

  • Cross-platform user-facing changes (Home feed news module and news screen - Brian)

Generated by :no_entry_sign: dangerJS against 7b5e09b4a382e5714e497ec64bf598c2e7e39382

ArtsyOpenSource avatar Jan 18 '24 09:01 ArtsyOpenSource

@brainbicycle I think we can close this one, thanks for the work here, as mentioned in the linked PR it was extracted and adapted 👍

araujobarret avatar Apr 04 '24 11:04 araujobarret