-
Notifications
You must be signed in to change notification settings - Fork 14
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
Schema 4.6 #193
Schema 4.6 #193
Conversation
Added the conversions for the values here: https://docs.google.com/spreadsheets/d/1XtOzZPSKRUoY1vKeH6yhX_3Jg6lfiNqJ6qQmXARunSI/edit?gid=0#gid=0 to fulfill the user story here: https://docs.google.com/document/d/1RllY4Q-_Ls9gHSfy9Iuwlctv0uV4E_S6Sg56rvkMt1o/edit?tab=t.0#heading=h.6kvpm1o6748w @ashwinisukale and @kaysiz , could you take a look? If the code is OK, then we're ready to merge and release bolognese. Thanks! |
@codycooperross thank you so much for finding the missing points from this, really appreciate it. Changes looks good to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not deep in this code, but it looks good to me, and passes all of the test suite.
Purpose
closes: #194
#195
#196