CombineCocoa icon indicating copy to clipboard operation
CombineCocoa copied to clipboard

Using private handleEvent API

Open mandarin6b0 opened this issue 2 years ago • 2 comments

Describe the bug I get an error while trying to upload a binary to Testflight

The app references non-public selectors in Payload/XXX.app/Frameworks/CombineCocoa.framework/CombineCocoa: handleEvent

To Reproduce Steps to reproduce the behavior:

  1. Include CombineCocoa (using cocoapods in my case)
  2. Archive and upload to Testflight

Expected behavior No error when distributing

Device:

  • XCode 14.0.1
  • CombineCocoa version 0.4.0

mandarin6b0 avatar Oct 06 '22 20:10 mandarin6b0

I have the exact same issue

StefanNestorov avatar Oct 09 '22 19:10 StefanNestorov

That's really interesting. handleEvent is the name of this method.

Could it be that Apple is confusing it with some private method? @mandarin6b0 @StefanNestorov Will you be able testing a branch if I give you one?

freak4pc avatar Oct 10 '22 17:10 freak4pc

@freak4pc sure

mandarin6b0 avatar Oct 20 '22 09:10 mandarin6b0

Can you try the branch handleevent-change please? @mandarin6b0

freak4pc avatar Oct 20 '22 09:10 freak4pc

@freak4pc I still get the problem. I found one more place with handleEvent. Should it be changed too?

mandarin6b0 avatar Oct 20 '22 09:10 mandarin6b0

Changed and apparently didn’t commit. Try now. On 20 Oct 2022, 12:23 +0300, Tien Nguyen @.***>, wrote:

@freak4pc I still get the problem. I found one more place with handleEvent. Should it be changed too? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

freak4pc avatar Oct 20 '22 10:10 freak4pc

Changed and apparently didn’t commit. Try now. On 20 Oct 2022, 12:23 +0300, Tien Nguyen @.>, wrote: @freak4pc I still get the problem. I found one more place with handleEvent. Should it be changed too? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.>

It's working now 👍

mandarin6b0 avatar Oct 20 '22 11:10 mandarin6b0

I'll merge it. THanks!

freak4pc avatar Oct 20 '22 11:10 freak4pc

Released in 0.4.1

freak4pc avatar Oct 20 '22 11:10 freak4pc

Thank you!On 20 Oct 2022, at 18:46, Shai Mishali @.***> wrote: Released in 0.4.1

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

mandarin6b0 avatar Oct 20 '22 11:10 mandarin6b0