Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@muhamiyan
Copy link
Contributor

Bismillah, should be to create a new quiz or new assignment via the add button.

Feature of Quiz and Assignment in courses provides the API endpoints /lms/quizzes/new and /lms/assignments/new so it worked fine, but this no longer works on the latest version of the LMS.

@pateljannat
Copy link
Collaborator

@muhamiyan This fix redirects the users to the list, but since the intention is to create a new quiz or assignment, they still need to click on the Create button. Can we send a query param from the redirectToForm function, such that if this parameter is present then we show the creation dialog directly?

@muhamiyan
Copy link
Contributor Author

thanks @pateljannat, showing the creation dialog is better, I will fix it.

@muhamiyan muhamiyan changed the title fix: remove redirect to create new quiz and new assignment fix: open dialog directly to create quiz and assignment Dec 9, 2025
@muhamiyan
Copy link
Contributor Author

Please to review, thanks @pateljannat

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@66c70dd). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1866   +/-   ##
==========================================
  Coverage           ?   19.95%           
==========================================
  Files              ?      156           
  Lines              ?     4817           
  Branches           ?        0           
==========================================
  Hits               ?      961           
  Misses             ?     3856           
  Partials           ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@cypress
Copy link

cypress bot commented Dec 9, 2025

lms    Run #1822

Run Properties:  status check passed Passed #1822  •  git commit 545b9ee8df ℹ️: Merge ae8ffd4cbd04a08f96c421a81c6900dfa5896a6a into 829245f3732eff551739d9c09c14...
Project lms
Branch Review issues-1
Run status status check passed Passed #1822
Run duration 01m 05s
Commit git commit 545b9ee8df ℹ️: Merge ae8ffd4cbd04a08f96c421a81c6900dfa5896a6a into 829245f3732eff551739d9c09c14...
Committer Apriliansyah Idris
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 2
⚠️ You've recorded test results over your free plan limit.
Upgrade your plan to view test results.
View all changes introduced in this branch ↗︎

@muhamiyan muhamiyan requested a review from pateljannat December 9, 2025 12:44
@pateljannat pateljannat merged commit 8de3996 into frappe:develop Dec 11, 2025
7 checks passed
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 2.42.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants