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
Stop showing cohort capacity #31433
Stop showing cohort capacity #31433
Conversation
… Remove logic restricting the cohort calculator to only display when a partner has specified their cohort size.
…ts has been deprecated, in favor of keyword arguments, and will be removed in Rails 5.1.
@@ -249,13 +249,13 @@ class Api::V1::RegionalPartnersControllerTest < ActionController::TestCase | |||
test 'show regional partner summer workshops for valid partner ID' do | |||
regional_partner = create :regional_partner_beverly_hills | |||
|
|||
get :show, partner_id: regional_partner.id | |||
get :show, params: {partner_id: regional_partner.id} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is to fix a deprecation warning while running tests in RubyMine
Using positional arguments in functional tests has been deprecated, in favor of keyword arguments, and will be removed in Rails 5.1.
capacity: null, | ||
loadingRegistrationCount: null, | ||
registered: null | ||
loadingEnrollmentCount: null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I change loadingRegistrationCount
to loadingEnrollmentCount
and registered
to enrolled
to match the function name and value returned from the server side.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Description
PLC-574
Remove "available seats" and "remaining capacity" rows from the cohort calculator
Remove logic restricting the cohort calculator to only display when a partner has specified their cohort size.
Remove "Cohort capacity csd" and "Cohort capacity csp" fields from the regional partner edit view and the regional partner info view
Testing story
Manual tests on local host.