Skip to content

Test PR | Turn on extracted xblock django setting flags#36334

Closed
farhan wants to merge 1 commit intomasterfrom
turn-on-extracted-xblocks
Closed

Test PR | Turn on extracted xblock django setting flags#36334
farhan wants to merge 1 commit intomasterfrom
turn-on-extracted-xblocks

Conversation

@farhan
Copy link
Contributor

@farhan farhan commented Mar 6, 2025

It's just a test PR to test extracted xblocks on the sandbox environment

@farhan farhan added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Mar 6, 2025
@farhan farhan self-assigned this Mar 6, 2025
@farhan farhan requested a review from kdmccormick March 6, 2025 12:44
@open-craft-grove
Copy link

open-craft-grove commented Mar 6, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

ttqureshi
ttqureshi reviewed Mar 7, 2025
View reviewed changes
lms/envs/common.py Outdated
# .. toggle_creation_date: 2024-11-10
# .. toggle_target_removal_date: 2025-06-01
USE_EXTRACTED_DISCUSSION_BLOCK = False
USE_EXTRACTED_DISCUSSION_BLOCK = True
Copy link
Member

@ttqureshi ttqureshi Mar 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to toggle Discussion Block flag or even keep it as we're not extracting it?

Copy link
Contributor Author

@farhan farhan Mar 7, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, It's fine, it's just for testing

Copy link
Member

@ttqureshi ttqureshi Mar 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok 👍

@kdmccormick kdmccormick force-pushed the turn-on-extracted-xblocks branch from 8a375e2 to 272773d Compare March 25, 2025 14:51
@open-craft-grove
Copy link

open-craft-grove commented Mar 25, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@kdmccormick kdmccormick self-assigned this Aug 1, 2025
@open-craft-grove
Copy link

open-craft-grove commented Aug 1, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Aug 4, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@farhan farhan force-pushed the turn-on-extracted-xblocks branch 2 times, most recently from 4dc5b32 to 5dd5bdf Compare August 12, 2025 11:56
@farhan
Copy link
Contributor Author

farhan commented Aug 12, 2025 •
edited
Loading

Update
I have rebased and resolved the conflicts coming from master.
Required settings flag are enabling from openedx/envs/common.py now

cc: @openedx/axim-aximprovements , @kdmccormick

@open-craft-grove
Copy link

open-craft-grove commented Aug 12, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Aug 13, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Sep 6, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Sep 9, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Nov 13, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@kdmccormick kdmccormick force-pushed the turn-on-extracted-xblocks branch from 19badda to 6aa6658 Compare December 16, 2025 16:08
@open-craft-grove
Copy link

open-craft-grove commented Dec 17, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Dec 17, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@kdmccormick kdmccormick removed the create-sandbox open-craft-grove should create a sandbox environment from this PR label Jan 30, 2026
@kdmccormick kdmccormick removed their request for review January 30, 2026 21:19
@farhan
Copy link
Contributor Author

farhan commented Mar 13, 2026

Closing the PR as it's not required anymore

@farhan farhan closed this Mar 13, 2026
@farhan farhan deleted the turn-on-extracted-xblocks branch March 13, 2026 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@ttqureshi ttqureshi ttqureshi left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@kdmccormick kdmccormick

@farhan farhan

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@farhan @open-craft-grove @ttqureshi @kdmccormick