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

Feature/override format option #8

Merged
merged 4 commits into from Nov 14, 2019
Merged

Feature/override format option #8

merged 4 commits into from Nov 14, 2019

Conversation

RossUK88
Copy link

Allow the package to have formats passed as a 3rd parameter.
This allows users to now specify a HTML format and get more accurate results from translating strings of HTML.

No breaking changes as this still defaults to text if nothing is passed in.

@introwit
Copy link
Member

@RossUK88 thank you for the contribution Ross ❤️ Can you please fix/add the necessary tests too? And also update the readme with the docs on what a 'format' actually is in the context of the package and what format options do what so that everyone can take advantage of your contribution 😄

Add additional test for taking a HTML parameter
Update README for Format parameter and options allowed, with link to Translate API Docs
@RossUK88
Copy link
Author

@introwit Should be all updated now. If you could have a quick check over that would be ace :)

@introwit introwit merged commit 1c9178b into JoggApp:master Nov 14, 2019
@introwit
Copy link
Member

@RossUK88 thank you so much Ross 🙂Since you added default param values to prevent any breaking changes, I tagged a new patch release

@RossUK88 RossUK88 deleted the feature/override_format_option branch November 14, 2019 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants