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

Modest updates to Readme.md #17

Merged
merged 9 commits into from
Nov 10, 2021

Conversation

Jerimee
Copy link
Contributor

@Jerimee Jerimee commented Oct 30, 2021

This pr is a baby step; it does only two things: correct a minor spelling typo in english and update readme.md

It is a branch based on develop NOT MASTER branch

This pr is a baby step; it does only two things: correct a minor spelling typo in english and update readme.md

It is a branch based on *develop* NOT MASTER branch
A humble bare-bones attempt at adding a service capable of serving embeds like this one - https://media.dlib.indiana.edu/media_objects/pg15bg11k

It barely works
add example, rename examples youtube examples, will add service specific examples later
@Jerimee
Copy link
Contributor Author

Jerimee commented Oct 30, 2021

note 4 commits - this PR should no longer be named "Very M" (apologies for my clumsiness)

@Jerimee Jerimee changed the title Very M Modest updates to Readme.md, add Indiana service Oct 30, 2021
@octfx
Copy link
Member

octfx commented Oct 31, 2021

Thank you very much for your contribution!

One general note: Could you split the readme changes and addition of a service into two separate pull request?
This way everything is clean and separated

@Jerimee
Copy link
Contributor Author

Jerimee commented Oct 31, 2021 via email

@Jerimee Jerimee changed the title Modest updates to Readme.md, add Indiana service Modest updates to Readme.md Nov 8, 2021
@Jerimee
Copy link
Contributor Author

Jerimee commented Nov 8, 2021

ok this is now just README and small typo

@octfx octfx merged commit 811d72e into StarCitizenWiki:develop Nov 10, 2021
@octfx
Copy link
Member

octfx commented Nov 10, 2021

Merged.

Thanks again for the readme updates!

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

Successfully merging this pull request may close these issues.

2 participants