Skip to content

Remove the quiz wizard - #1702

Open
bpfoley wants to merge 1 commit into
DistributedProofreaders:masterfrom
bpfoley:rm-wizard
Open

bpfoley wants to merge 1 commit into
DistributedProofreaders:masterfrom
bpfoley:rm-wizard

Conversation

@bpfoley

@bpfoley bpfoley commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

It was only intended for use by site admins, and is PHP4-era code that would need a lot of work to bring up to our current security and code quality standards.

It was only intended for use by site admins, and is PHP4-era code that
would need a lot of work to bring up to our current security and
code quality standards.
@cpeel
cpeel self-requested a review October 27, 2026 20:51

@srjfoo srjfoo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am impressed that the wizard did as good a job as it did in creating the quizzes, but both the code itself, and the code that it creates are badly outdated, and we haven't created new quizzes in years.

We have enough existing quizzes that, if we need to create new ones, we can use existing ones as templates and develop instructions on how to do it. I like the idea of simplifying our code!

@cpeel cpeel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think removing this is the right thing to do and removes dead/dying code from the codebase.

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.

3 participants