Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Extend customization - #585

Closed
ramlmn wants to merge 1 commit into
refined-github:masterfrom
ramlmn:add-options
Closed

Extend customization#585
ramlmn wants to merge 1 commit into
refined-github:masterfrom
ramlmn:add-options

Conversation

@ramlmn

@ramlmn ramlmn commented Jul 7, 2017

Copy link
Copy Markdown
  • Add page specific options (including tab-size)
  • Fetch real-name with GitHub api if access token is available (needs review)

@sindresorhus @bfred-it This PR needs some review. WDYT?

@sindresorhus

sindresorhus commented Jul 7, 2017

Copy link
Copy Markdown
Member

From the readme:

We're happy to receive suggestions and contributions, but be aware this is a highly opinionated project. There's a single commonly-requested option but we're not interested in adding more as it's a slippery slope into adding one for everything. Users will always disagree with something. That being said, we're open to discussing things.

Probably better to open issues about this in the future as we're unlikely to add options unless it's something that is highly contentious.

@fregante

fregante commented Jul 7, 2017

Copy link
Copy Markdown
Member
  1. We're not gonna implement both the API call and the normal fetch.
  2. Options need to be discussed beforehand.

@ramlmn

ramlmn commented Jul 7, 2017

Copy link
Copy Markdown
Author

Works for me 😊. You may close this PR if its so.

Comment thread src/libs/show-names.js
export default async () => {
const myUsername = getUsername();
const cache = (await getCachedUsers())[storageKey];
const cache = (await getCachedUsers())[storageKey] || {};

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please send this line as a separate PR so we can get that in ASAP :)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment thread src/content.js
const username = getUsername();

markUnread.setup();
// Things that we want to do only when the user is logged in

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this. Separate PR? 😃

@ramlmn
ramlmn deleted the add-options branch July 8, 2017 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants