receive_sharing_intent icon indicating copy to clipboard operation
receive_sharing_intent copied to clipboard

Use customizable app group id

Open Klabauterman opened this issue 5 years ago • 5 comments

With these changes you can define your own app group id instead of group.BUNDLE_ID by putting it into the Info.plist. If no AppGroupId is found in the Info.plist file it will use the old convention instead.

(I transferred my app from one apple account to another and lost my app group id by that. So the old approach of this repo wasn't working anymore.)

Klabauterman avatar Mar 29 '20 01:03 Klabauterman

@KasemJaffer I highly recommend to look at this and try to merge in into master :-) There are some use-cases for the group id not being the same as group.bundle_id. I tested this (I had to because of our architecture, I cannot simply use group.bundle_id.

@Klabauterman maybe you could squash the commits (at least the last 2 into the example adaptation). Why did you remove the test code?

GRBurst avatar Apr 20 '20 09:04 GRBurst

It would be very great, if this PR could be merged.

EduardFrank avatar Apr 27 '20 16:04 EduardFrank

Hey guys, I am sorry, i totally forgot about my PR. I'll check your comments today.

Klabauterman avatar Jun 13 '20 11:06 Klabauterman

@GRBurst commits are squashed, readme corrected, intend fixed, example code changed

Klabauterman avatar Jun 14 '20 15:06 Klabauterman

Hey @Klabauterman, I'm still very thankful since I use your PR in my app for quite some time now :+1:

I think your last commit (0ede807) should be on a different branch since it does not directly relate to the PR.

@KasemJaffer Again I'd really appreciate if you could consider to merge this. It really solves some real world issues. Please make a statement whether or not you are going to merge it and what issues you see towards merging it ;-)

GRBurst avatar Jan 15 '21 19:01 GRBurst

Thank you all for the great work. I'm closing this because i merged this other PR which has the same functionality. I will be releasing v1.5.0 as soon as possible after i run some testes.

KasemJaffer avatar Jan 18 '24 17:01 KasemJaffer