Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings
This repository was archived by the owner on Nov 1, 2017. It is now read-only.

Makeing it clear where the presented code goes#820

Closed
edusantana wants to merge1 commit intogithub:masterfromedusantana:patch-1

Conversation

@edusantana
Copy link

The guide doesn't show where the presented code goes.

@kdaigle
Copy link
Member

I'm not exactly clear on what "code" you're talking about here. Are you looking for some language to state that you should check the scopes after you receive theaccess_token?

@edusantana
Copy link
Author

When I was reading it I didn't know where to put the fowling code:

#!ruby# check if we were granted user:email scopescopes=JSON.parse(result)['scope'].split(',')has_user_email_scope=scopes.include?'user:email'

It goes right AFTER the line:

access_token = JSON.parse(result)['access_token']

@edusantana
Copy link
Author

I have added the text so that others would know it, but feel free to reject if it's not a good ideia.

@kdaigle
Copy link
Member

@edusantana ah, now I see what you mean. 😄 I'll update the docs by clarifying where that code should live by changing the actual code block. I'm going to close this PR as I'd like to document this differently than you have but thanks so much for the suggestion!

@jasonrudolph
Copy link
Contributor

@edusantana: Thanks for letting us know that this part of the docs needed improvement. We've updated the docs in#829.

jasonrudolph pushed a commit that referenced this pull requestJan 23, 2016
Reinsert missing definition lists
Sign up for freeto subscribe to this conversation on GitHub. Already have an account?Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@edusantana@kdaigle@jasonrudolph

[8]ページ先頭

©2009-2025 Movatter.jp