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 strict argument added #16

Merged
merged 4 commits into from
Nov 10, 2023

Conversation

vgavro
Copy link
Contributor

@vgavro vgavro commented Nov 7, 2023

fix: strict argument added (defaults to true as in json.loads for compatibility)

Usecase: consider json below returned by chatgpt, fixer can't handle it without strict=False parameter.

{\n  "text": "Apart from the Business Unlimited Pro 2.0 Smartphone package, we also offer the Business Unlimited Tablet Start plan. Here are the details:\n\n# Business Unlimited Tablet Start\nPrice: $30 per month per tablet\n\nFeatures:\n- 5G Nationwide/4G LTE\n- Video streaming at 480p\n\nThis plan also offers the option to add additional devices. Please let me know if you need information on any other plans or services.",\n  "quick_replies": ["Tell me more about additional devices", "What are the terms and conditions?", "Do you offer any discounts?", "I want to know about other plans"]\n}'

@alingse alingse merged commit e30bd7e into half-pie:master Nov 10, 2023
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