David Ormsbee

Results 56 comments of David Ormsbee

Ack, sorry, forgot to merge this. But will do so once checks finish running.

😞 Yeah, we can't break OLX as it's being used in the wild, and inline definitions like this are too pervasive at this point to deprecate. So that ability needs...

@Agrendalath: What currently happens if the configured `default_class` isn't found (e.g. it's an import error)? Can we make edx-platform code fall back to a hardcoded default in that case (and...

@Agrendalath: I'm okay without the final check. Please just schedule with @jristau1984 as usual. That being said, if this change is going to break dev environments when people pull the...

@regisb: I think we should be careful to distinguish between changing the connection settings and converting existing table data. We can change the connection settings without rebuilding tables or changing...

@regisb: I'm talking with folks at 2U about switching over the Studio connection initially, followed by LMS if that goes smoothly, hopefully in the next couple of weeks. Migrating existing...

Thank you so much for taking on this important work! > While working on a fix for this issue, I upgraded the MySQL character_set and collation in Tutor-nightly to utf8mb4...

Also, I'm not sure how we want to handle the migrations of really big tables for the community. Converting something like `courseware_studentmodule` may cause significant downtime for large sites.

As background, the explicitly created encodings/collations for `content_staging` and the `oel_` apps use `openedx_learning` helper functions to define their fields: * [`case_insensitive_char_field`](https://github.com/openedx/openedx-learning/blob/main/openedx_learning/lib/fields.py#L38-L63) * [`case_sensitive_char_field`](https://github.com/openedx/openedx-learning/blob/main/openedx_learning/lib/fields.py#L68-L91) * [`key_field`](https://github.com/openedx/openedx-learning/blob/main/openedx_learning/lib/fields.py#L112-L123) Those use the `MultiCollationMixin`...

Also, we need to summarize this work in the [operator release notes for Redwood](https://openedx.atlassian.net/wiki/spaces/COMM/pages/3890380898/Next+Release+Redwood) before the release goes out.