Chris Patti

Results 228 comments of Chris Patti

Got past that, and successfully build xml for toggle flags in redwood! however, we are blocked in sumac.master, sphinx hangs forever: ``` ~/src/openedx/edx-platform/docs (open-release/sumac.master*) » make xml [.venv] feoh@prometheus sphinx-build...

Well it ran for a few hours and then got tactically nuked by Linux's OOM killer. :(

@pdpinch That's a great idea but I'm not sure I can properly King Solomon the configuration without chopping out the good bits. I'll do some more staring at the configs...

Cherry picked this into my local sumac branch and it got past the OOM killer blockage point! As you say build still takes a VERY long time so it's at...

@farhaanbukhsh Who should I talk to please about getting this backported to sumac? Happy to do it myself if there's a process I can follow. Thanks again very much for...

With Maria's help I came up with the following list of additions and deletions for Sumac for feature/toggles: ``` -------------------------------------------------------------------------------------------------------------------------------------------------------------------------- ~ » cat /mnt/c/Users/feoh/Downloads/settings-diff | grep '^[+-]' | grep "

Added more feature flags. Couldn't easily find the following toggles. Will need to search other repos maybe? ``` N N N N ```

Thanks so much that's incredibly helpful!

The following toggles/features have been removed in Sumac. Do any of these require further documentation? COURSEGRAPH_DUMP_COURSE_ON_PUBLISH FEATURESENABLE_LIBRARY_AUTHORING_MICROFRONTEND FEATURESENABLE_V2_CERT_DISPLAY_SETTINGS commerce.transition_to_coordinator.checkout commerce.transition_to_coordinator.refund contentstore.library_authoring_mfe discussions.enable_reported_content_notifications learner_dashboard.enable_b2c_subscriptions notifications.enable_coursewide_notifications notifications.enable_notifications_filters notifications.enable_ora_staff_notifications notifications.show_notifications_tray studio.enable_course_update_notifications BLOCKSTORE_BUNDLE_CACHE_TIMEOUT BUNDLE_ASSET_STORAGE_SETTINGS...

This is all set. Merged just now by @sarina - thanks for all the help!