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

fix Open Graph og:image link #6612

Merged
merged 4 commits into from
Apr 14, 2019

Conversation

ashimokawa
Copy link
Contributor

This fixes the Open Graph og:image link, which fix fix twitter preview cards, among others

you can see the problem by using this link, and then clicking on the "Open Graph" button:

http://debug.iframely.com/?uri=https://try.gitea.io

Or use the twitter card validator

https://cards-dev.twitter.com/validator

Corresponding Codeberg issue: https://codeberg.org/Codeberg/Community/issues/39

@codecov-io
Copy link

codecov-io commented Apr 14, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@2342d2d). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #6612   +/-   ##
=========================================
  Coverage          ?   40.47%           
=========================================
  Files             ?      406           
  Lines             ?    54484           
  Branches          ?        0           
=========================================
  Hits              ?    22054           
  Misses            ?    29393           
  Partials          ?     3037

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2342d2d...f528eb2. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 14, 2019
templates/base/head.tmpl Outdated Show resolved Hide resolved
@lunny lunny added the type/bug label Apr 14, 2019
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 14, 2019
@jonasfranz jonasfranz requested a review from lafriks April 14, 2019 10:57
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Apr 14, 2019
@techknowlogick
Copy link
Member

@ashimokawa Please check "allow edits from maintainers" so we can merge latest changes from master before merging.

@techknowlogick techknowlogick added this to the 1.9.0 milestone Apr 14, 2019
@ashimokawa
Copy link
Contributor Author

@ashimokawa Please check "allow edits from maintainers" so we can merge latest changes from master before merging.

Done,
also rebased to latest master, feel free to squash the two commits

@techknowlogick techknowlogick merged commit de6527b into go-gitea:master Apr 14, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants