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

[android][sample] Disable Fresco integration for now #5101

Closed
wants to merge 2 commits into from
Closed

Conversation

passy
Copy link
Member

@passy passy commented Aug 31, 2023

[android][sample] Disable Fresco integration for now

Summary:
There's currently a startup crash

https://gist.github.com/passy/3f6b5935d8f9171ea0d3d915857b9786

We'll need to wait for a new Fresco release. In the meantime, let's comment this out so we can still use the rest of the app.


Stack created with Sapling. Best reviewed with ReviewStack.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 31, 2023
@facebook-github-bot
Copy link
Contributor

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@github-actions
Copy link
Contributor

github-actions bot commented Aug 31, 2023

Pull Request Test Coverage Report for Build 6035737025

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 50.879%

Totals Coverage Status
Change from base Build 6029793696: 0.0%
Covered Lines: 8413
Relevant Lines: 15351

💛 - Coveralls

@facebook-github-bot
Copy link
Contributor

@passy merged this pull request in 42b95c2.

passy added a commit that referenced this pull request Sep 19, 2023
facebook-github-bot pushed a commit that referenced this pull request Sep 19, 2023
Summary:
Back out "Disable Fresco integration for now (#5101)"

Original commit changeset: 42b95c2

This is now working again with Fresco 3.1.0

 ---
Stack created with [Sapling](https://sapling-scm.com). Best reviewed with [ReviewStack](https://reviewstack.dev/facebook/flipper/pull/5159).
* #5160
* __->__ #5159
* #5158

Pull Request resolved: #5159

Reviewed By: antonk52

Differential Revision: D49411925

Pulled By: passy

fbshipit-source-id: 34a422e60102de56fb6588e05cf4729a30e0e112
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants