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

Add OpenGraph Support #711

Merged
merged 4 commits into from
Jul 7, 2020
Merged

Add OpenGraph Support #711

merged 4 commits into from
Jul 7, 2020

Conversation

Daltz333
Copy link
Member

Closes #708

source/conf.py Outdated Show resolved Hide resolved
@sciencewhiz
Copy link
Collaborator

sciencewhiz commented Jul 7, 2020

Right now, with this PR, it isn't enough for chiefdelphi to make a preview using the OpenGraph data (but I'm not sure what is necessary). I'm guessing it's the image attribute.

source/conf.py Outdated Show resolved Hide resolved
Daltz333 and others added 2 commits July 6, 2020 23:23
Co-authored-by: Austin Shalit <austinshalit@gmail.com>
@sciencewhiz
Copy link
Collaborator

The image has a transparent background. It would probably be best to have a non-transparent version for this, so the white text in WPI and FRC doesn't get lost.

@Daltz333
Copy link
Member Author

Daltz333 commented Jul 7, 2020

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Daltz333 Daltz333 merged commit df486fa into wpilibsuite:master Jul 7, 2020
@Daltz333 Daltz333 deleted the opengraph branch July 30, 2020 14:53
Starlight220 pushed a commit to Starlight220/frc-docs that referenced this pull request Aug 27, 2020
* Add Opengraph Extension

* Use main site name

* Update source/conf.py

Co-authored-by: Austin Shalit <austinshalit@gmail.com>

* Use WPILib logo

Co-authored-by: Austin Shalit <austinshalit@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add rich preview metadata
4 participants