-
Notifications
You must be signed in to change notification settings - Fork 375
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
SHAREKIT 2.0 – ONE MAIN GLOBAL FORK #283
Comments
Best I can tell I have all of the common changes in my fork, as well as fixes for leaks, allow configurable sharers list (so that you can subclass SHKsharers), callbacks for canceling out of sharers, added UI for flickr and removed a round trip to the server to set item attributes, allow you to nobble offline and other enhancements. Off the top I've merged flickr integration, delegate config, new facebook and many more. I would be open to having other admins on my fork, and could spare a little time to merge changes, but I don't want to be the only one. -s |
This is really good! I see you're very active last time. Thank for the quick reply. The main problem for ShareKit newbie is to select right fork. Then when any bug appears, e.g. Facebook doesn't share text right now, people try to fix and it's hard to find fixes and how to embed it in own changes code and so on. This becomes a mission impossible for newbies. So that's why I believe we need one fork and maybe ask @ideashower to delegate rights to new lead or so. Honestly I am not very experienced in ShareKit, I just see my own problem and suggest that it's common. Let's wait what other guys think. |
I agree with you that there should be one global fork. Apperently @ideashower doesn't have the time to maintain this fork. If you guys need some help getting ShareKit to V2.0 I am willing to contribute to this project. |
This situation exemplifies the strength and weakness of open libs shared via GitHub. I suspect that ideashower is simply too busy to manage all the work that community is contributing and I suspect that would be the case for any individual or group who would take over. I would recommend that any beginner start with a fork of the jayway fork (https://github.com/jayway/ShareKit) and merge in other forks as needed. |
Hey there- interested to see ShareKit in the hands of the community as well. A little busy these days with projects and usually use Github to keep the changes / additions somewhere at least. A central repository on Github would be great. What is the next step to continue? |
I think this is a great idea. I sent message to @ideashower 5 months ago suggesting this and never got a response back. I've created a GitHub organization called ShareKit and pushed the current contents of the repo there. I have no problem giving others admin rights to the org. I think the next step should be deciding on a fork that will become the seed of ShareKit 2.0. It might also be useful to port the current issues from ideashower to the new org. |
I've spent a fair amount of time merging all of the best changes into my fork. I've even fixed some long standing memory leaks and added enhancements to contributed elements. From: "scompt" reply@reply.github.com I think this is a great idea. I sent message to @ideashower 5 months ago suggesting this and never got a response back. I've created a GitHub organization called ShareKit and pushed the current contents of the repo there. I have no problem giving others admin rights to the org. I think the next step should be deciding on a fork that will become the seed of ShareKit 2.0. It might also be useful to port the current issues from ideashower to the new org. https://github.com/ShareKit/ShareKit Reply to this email directly or view it on GitHub: |
I'd like to contribute, with the caveat that a) I still haven't grokked git and am therefore of limited use as an admin and b) I'm completely new to ShareKit.
Unless there's a good argument against either of these next steps...? -C |
P.S. One thing that surprises me is that the ShareKit/ShareKit network graph is completely disconnected from the original. It seems like that might make it harder to pull useful commits from all of the existing project forks. (?) |
There are a ton of forks that have become less and less compatible. I think that my fork as the bulk of the changes that people have been looking for and some fixes to fundamental issues. |
@clozach That's true. I pulled locally and then pushed up to github, which blew away the network graph. The one benefit that it has is that it doesn't say 'forked from ...', which might lend the repo some credibility as the canonical one. What are the others' opinion about this? Fork with network graph or pull/push without network graph? Let's go with @troppoli 's repository in any case. That'll get us up and running with a good base from which we can start looking at the existing tickets to see which should be ported. |
If you pull and push, you loose the history of where changes came from, what changes are in and the comments from the checkins. I think that they are too valuable to loose. From: "scompt" reply@reply.github.com @clozach That's true. I pulled locally and then pushed up to github, which blew away the network graph. The one benefit that it has is that it doesn't say 'forked from ...', which might lend the repo some credibility as the canonical one. What are the others' opinion about this? Fork with network graph or pull/push without network graph? Let's go with @troppoli 's repository in any case. That'll get us up and running with a good base from which we can start looking at the existing tickets to see which should be ported. Reply to this email directly or view it on GitHub: |
You can have both. http://gbayer.com/development/moving-files-from-one-git-repository-to-another-preserving-history/ This was the first google hit, I am sure there are other ways too. @troppoli, it needs to be off of your personal fork to get any legs here as an "official" 2.0 |
I've never done that but it looks promising. On Aug 31, 2011, at 8:24 PM, coneybeare reply@reply.github.com wrote:
|
I've re-pushed the ShareKit/ShareKit repository using the method described in @coneybeare 's linke (essentially clone/push instead of pull/push). The history is there, but the network isn't. Is that enough to go on? I used @troppoli 's fork this time. @clozach I've just created the wiki for the repository. Let's stick the documentation in there. You should be able to edit it. |
Count me in, people. I should note that I've already added iOS[REDACTED] support for SHKTwitter. |
@coneybeare well I just pushed to sharekit/troppoli and tried to open a pull request to sharekit/sharekit... not sure how to do it because of the way it was created. Any ideas? |
I am not entirely sure, but because ShareKit/Sharekit has your recent code, I think it would be easiest to create a new branch off of that, re-apply the changes that you just pushed to sharekit/troppoli, then submit a pull request back up to ShareKit/ShareKit. @scompt should make you a collaborator on the main repo so you can accept your own pull requests and push to master there |
Which commit added the iOS[REDACTED] support? |
@troppoli I've just deleted the repo again and forked it using the normal methods. Try again with your pull request. I've also added you so that you can push to the repository. @coneybeare Your issue got deleted when deleting and forking the repo. Can you add it again? |
As it turns out, the new fork has my latest change, so I can't initiate a pull, but I will on the next change. From: "scompt" reply@reply.github.com @troppoli I've just deleted the repo again and forked it using the normal methods. Try again with your pull request. I've also added you so that you can push to the repository. @coneybeare Your issue got deleted when deleting and forking the repo. Can you add it again? Reply to this email directly or view it on GitHub: |
@coneybeare I've just enabled issues for the repository. I think the rest of the project management stuff is also enabled with that feature. Do you notice anything else that is missing? That means the only downside is the lack of credibility. Personally, I think it would be better if it wasn't forked, but rather cloned as it was previously. I think the credibility issue is also important. However, I'm not sure what problem @troppoli had when trying to make his pull request and if it has to do with the way the repo was created. |
The problem was that @troppoli did not reclone from the sharekit/sharekit repo, then do the pull request. He tried to do the request across repo's. It has to be done as before, with you pulling the history from him, then him doing the process outlined in #283 (comment) A new beginning, a 2.0 for share kit, really needs to be a fresh start, pulling from an up-to-date repo like @troppoli's to start with. I won't contribute to yet another fork based off of the 1.0, and I am sure there are many like me. |
Guys, thanks everybody to be active! Sorry for late, I was on a vocation. – have you decided to organize a core team or work in Git style with personal forks and pull requests? |
@alondon: Not pushed yet, as the code is still under NDA and can't be made public. |
@coneybeare OK, that makes sense. I've re-done the repo using clone/push. Hopefully everything works now. Sorry to everyone who's tried to watch/comment/whatever on the repo while I've been getting this sorted out. @SlavaBushtruk I've added you to the new repo. I'd be more for the git development style with personal forks. Shall we move the discussion over to the new repo? I've just added a 2.0 milestone. Let's make a release! |
Thanks! So the best way to push changes is to make a new fork and the make pull request for you? |
I made 3 issues in new repo to discuss. Just not sure how to notify you about it except this topic. |
I agree that development should be done in personal forks with pull requests as a primary method of integrating fixes. At the same time, a core team should have pull/push privileges on github.com/ShareKit, with at least 2-3 full owners. Not would more members reduce risk to the community at large, but it's more fair to the org. members since verifying pull requests, managing issues prioritization, and defining stable releases can be a lot of work! @scompt, I'm sure you want to make sure that there aren't too many cooks in the kitchen, but please at least invite @troppoli in (if he wants), and me and/or @coneybeare (if you trust us) so that we can share the load. :) I'm cramming as much git/github info as I can fit into my tiny brain right now in order to become a useful resource on this project.... |
I've added the following users as owners to the ShareKit/ShareKit repository. I think that should be a good start at a core team. We can make changes later if the need arises. |
It's great that you all are trying to resurrect this project but my current project timeline is tight, so I'm wondering - is there a fork out there where at least the FBConnect portion has been updated to use FB's latest API? Thanks! |
@benjaminjackson Was just curious whether the fix was a backwards compatible one that could be made with iOS 4.x. Figured it out anyway. |
Hello guys, could someone please explain from where I should get troppolis fork? From https://github.com/troppoli/ShareKit or from here? If from here, where do I find it? The files from https://github.com/ideashower/ShareKit are all from 2010. Question 2 while I'm at it, does troppolis fork work with Thanks for any help, lexar |
troppoli's fork is at https://github.com/troppoli/ShareKit |
@troppoli I am sure you are aware of this but due to some changes of modal view controllers in iOS 5, the current sharekit has major issues when compiled with xcode 4.2 and base sdk of 5.0. Cancel buttons will not dismiss the current view when closing out emails, or Twitter. I was able to fix the twitter part by removing the SHKCUrrenthelper and replacing with self dismissmodalviewcontroller code, but not on the email. Have you been able to work on a good fix for this issue? |
See the iOS 5 documentation for the new method presentingViewController and changes to parentViewController as it is now out of NDA. parentViewController @Property(nonatomic, readonly) UIViewController *parentViewController Prior to iOS 5.0, if a view did not have a parent view controller and was being presented modally, the view controller that was presenting it would be returned. This is no longer the case. You can get the presenting view controller using the presentingViewController property. Availability presentingViewController @Property(nonatomic, readonly) UIViewController *presentingViewController Availability |
Yes, I realize and have read this documentation. My question is about ShareKit...on iOS 5 because of the changes you wrote above, the view for logging into Twitter to authorize the app, does not disappear upon authorizing, or clicking the cancel button. Where in the ShareKit code needs to be changed in order to have that disappear? |
@spiderguy84 We're working on a patch for this problem over here if you're interested in switching the fork you're using. |
I was able to take care of this problem with the help of someone in iphonedevsdk.com. One thing I am having issue with now is Facebook not sharing text. I use this code:
}[/CODE] |
Ok, here is what I am having troubles with right now on the fork I am using. When you share a URL, the facebook window doesn't show the link that is being shared, although it does end up getting posted. I'm concerned that the user will think something is wrong when they don't see it, and cancel it. How can I set it up to show in the preview window of the app? Here is what the user sees: Here is what gets posted: |
I've updated my files from the ShareKit web site with those in here. Facebook has major issues. When i press FB share, it pops app the facebook app, and do nothing more. |
evripidis, this problem occurs, when your application ID (which you registered in facebook) is incorrect. |
I'm having difficulties following this (I'm not that into Git yet I guess). Will there be a ShareKit 2.0 in the coming days or weeks or even months? |
when i am sharing, in sharekit 2.0 LINK, everything is working ok, |
@KanybekMomukeyev The app ID is correct cause i can see the ID info into the FB app. Here is exactly where i stuck. I am sharing a text with Link and nothing happens |
@sebrk at the moment the development branch is the most recent: Here are the most up to date installation instructions (if I'm not mistaken): |
My THANK TO YOU ALL CONTRIBUTORS post — http://www.alterplay.com/ios-dev-tips/2011/12/open-source-sharekit2.html |
I got the error like operation could not be completed, contents ={ "OAuthException"} contents ={ "(#100) action links must have string 'href' and 'text' attributes"} Can any one help on this |
Please: Why the sharekit added to my project is not working properly?Example: In SHKActionSheet.m files inside |
Hi, |
Some vendors support SSO login. is it possible to add the feature in ? |
Guys, I think it's time for ShareKit 2.0 managing only by community. So much good code was written, so much good features and fixes. That's really good!
But when I try to gather all these pieces from different forks .. it's just impossible.
We really need one common fork and mentors who will take care of pull requests.
Let's discuss?
The text was updated successfully, but these errors were encountered: