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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

utterances 1.0 #25

Merged
merged 4 commits into from Mar 26, 2018

Conversation

Projects
None yet
1 participant
@jdanyow
Member

jdanyow commented Mar 26, 2018

  • Migrate build system to parcel 馃摝 馃殌

  • Switch from a classic GitHub OAuth App to the newer GitHub App authentication model.

    鈿狅笍 Site owners must install the Utterances GitHub App on the GitHub repo that hosts their comments.

    The change was required for utterances to take advantage of the more granular permissions scheme offered by the GitHub App model. With the current OAuth App, utterances must request the public_repo scope just to read/write issues and comments. A lot of extra permissions we don't need come along with this scope and users have raised issues (see #23). With the new app model, utterances will only have read/write permissions on issues & comments in repos where the utterances app is installed.

  • Inlines README.md during the build.

    Previously the html for the README.md file was loaded via a GitHub API call and inserted into the utteranc.es home page at run-time. This was a lazy hack, now that we're using parcel+posthtml it can be integrated in the build very easily.

jdanyow added some commits Mar 26, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment