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

Completion callback improvements #1

Merged
merged 1 commit into from
Mar 6, 2014
Merged

Conversation

virl
Copy link

@virl virl commented Mar 6, 2014

Sharing completion block called with success only upon real sharing completion (when content is really posted to social network).
Fixed Mail and Messages image attachments.
Using PNG attachment where possible.
Fixed activityType names for non-system activities.

…ompletion (when content is really posted to social network).

Fixed Mail and Messages image attachments.
Using PNG attachment where possible.
@huguesbr
Copy link
Owner

huguesbr commented Mar 6, 2014

Hey @virl I think you've pull the wrong repo.. I'm already forking the original repo. you should submit to the original repo. But you're request seem awesome!
https://github.com/romaonthego/REActivityViewController

@huguesbr huguesbr closed this Mar 6, 2014
@virl
Copy link
Author

virl commented Mar 6, 2014

@huguesbr anyway, please accept this pull request, because original repo still not accepted your own pull request.

@huguesbr huguesbr reopened this Mar 6, 2014
@virl
Copy link
Author

virl commented Mar 6, 2014

I've created pull request to original repository, but please accept to yours too.

huguesbr added a commit that referenced this pull request Mar 6, 2014
Completion callback improvements
@huguesbr huguesbr merged commit b7a1350 into huguesbr:master Mar 6, 2014
@huguesbr
Copy link
Owner

huguesbr commented Mar 6, 2014

got it.. accepted

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants