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

Skip to content

Conversation

@pateljannat
Copy link
Collaborator

  1. Users with role Batch Evaluator can now edit batches created by them.
  2. When a batch is unpublished, the previous logic used to check if the user has a student role, they were redirected to the batch list. But a user may have both student and evaluator role, in which case evaluators were unable to access unpublished batches. Now the logic instead checks if the user is not moderator and not evaluator before redirecting.
  3. On a new site, signup is disabled by default, but the LMS Settings which controls signup was not disabled. This caused inconsistency in behaviour.
  4. Job application list only showed the first 20 results and Load More button was missing. Added that.

@pateljannat pateljannat marked this pull request as ready for review December 2, 2025 11:02
@cypress
Copy link

cypress bot commented Dec 2, 2025

lms    Run #1784

Run Properties:  status check passed Passed #1784  •  git commit 197fc7df33 ℹ️: Merge 34e07b0083ce93b9120538756fbde7021c83746f into 1c0be8a2ec642ce2e8051effea77...
Project lms
Branch Review issues-153
Run status status check passed Passed #1784
Run duration 00m 55s
Commit git commit 197fc7df33 ℹ️: Merge 34e07b0083ce93b9120538756fbde7021c83746f into 1c0be8a2ec642ce2e8051effea77...
Committer Jannat Patel
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 ↗︎

@pateljannat pateljannat merged commit 0107032 into frappe:develop Dec 2, 2025
6 checks passed
@frappe-pr-bot
Copy link
Collaborator

🎉 This PR is included in version 2.41.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.

2 participants