Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: misc batch issues #667

Merged
merged 1 commit into from
Oct 27, 2023
Merged

fix: misc batch issues #667

merged 1 commit into from
Oct 27, 2023

Conversation

pateljannat
Copy link
Collaborator

  1. Fixes billing multicurrency issue.
  2. Archived and Enrolled batches will now be displayed in the descending order of start date.
  3. If the billing country is India, then the billing state will be mandatory and there will be a validation to check the correctness of the state.

@cypress
Copy link

cypress bot commented Oct 27, 2023

Passing run #345 ↗︎

0 1 0 0 Flakiness 0

Details:

Merge a49871c into e400579...
Project: lms Commit: b27b12fc1d ℹ️
Status: Passed Duration: 01:20 💡
Started: Oct 27, 2023 10:43 AM Ended: Oct 27, 2023 10:44 AM

Review all test suite changes for PR #667 ↗︎

@pateljannat pateljannat merged commit 50fbe00 into frappe:main Oct 27, 2023
2 of 3 checks passed
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.

1 participant