Skip to content
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

Prepare master for v5 release. #551

Merged
merged 15 commits into from
Jun 1, 2019
Merged

Prepare master for v5 release. #551

merged 15 commits into from
Jun 1, 2019

Conversation

stefanbuck
Copy link
Member

@stefanbuck stefanbuck commented May 4, 2019

The v5 branch contains all the latest major changes needed for the next major version

@stefanbuck stefanbuck force-pushed the v5 branch 4 times, most recently from d836042 to 75de802 Compare May 6, 2019 21:13
@stefanbuck stefanbuck force-pushed the v5 branch 2 times, most recently from 688849f to 8aaf7e7 Compare May 23, 2019 21:52
stefanbuck and others added 15 commits June 1, 2019 23:13
* Remove lazy loader for relative files

* Add setting to enable support for private repositories

* Insert real href tag

* Add token field to settings page

* Add octolinker api helper

* Align format with new octolinker api

* Use ZEIT now deployment

* Split head from base sha in url

In some cases the file path in PR comment contains head and a base sha like https://github.com/user/repo/pull/123/files/f18370bde83d7f1ced7acf5af54a21d2e53912ff..637d21dd86c8f50e9100b8da656e43cadcccde4d#diff-fd9ad2ea8184b3a5e7d76f54e87ae438

* Resolve urls with either the github or octolinker API

* Update E2E test to work with href attribute

* Correct grammar

See #543 (comment)

* Implement simple GitHub search

* Remove option to toggle line indicator

* Open settings dialog rather than being a new tab

* Fix GitHub API call in Firefox

* Simplify loader

* Add trusted url resolver to reduce load on octolinker api

* Fix tests
@stefanbuck stefanbuck merged commit 0174f49 into master Jun 1, 2019
@stefanbuck stefanbuck deleted the v5 branch June 1, 2019 21:17
@jeremy-ww
Copy link

@stefanbuck Which APP did you use to make this picture? ./demo.png

@stefanbuck
Copy link
Member Author

https://www.figma.com/

@jeremy-ww
Copy link

Thanks :)

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

Successfully merging this pull request may close these issues.

2 participants