Skip to content

Update gist posting to work with the lastest changes to gist.github.com #104

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

Open
wants to merge 4 commits into
base: experimental
Choose a base branch
from

Conversation

nilbus
Copy link

@nilbus nilbus commented Aug 10, 2011

Using a more generic regex to find the gist id. The old regex was failing because github no longer considers this HTTP request to be logged in, despite having the API token, so it does not provide an edit button.

Github now expects the parameter action_button=private, instead of private=true. All gists were being created as public gists.

The "TODO: Replace true with private preference" comment was outdated, since this has since been done.

Note: This patch does not require GitX to be rebuilt. The updated history.js file can be dropped into an existing GitX installation at /Applications/GitX.app/Contents/Resources/html/views/history/history.js and it will go into effect after restarting GitX. The GitX download for SciMed/gitx contains this patch.

nilbus and others added 4 commits August 10, 2011 11:06
No authentication support yet; that will require using OAuth2 tokens.
Better logging.
Keep multiple -s from showing up in patch filenames.
Using cookies for token storage.
Made the gist description more readable.
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.

1 participant