search

department-of-veterans-affairs / veterans-employment-center

Veterans Employment Center
https://www.vets.gov/veterans-employment-center
5 stars 4 forks source link

Comma separate 'Special commitments to'. #392

Closed U-DON closed 7 years ago

U-DON commented 7 years ago

Resolves department-of-veterans-affairs/vets.gov-team#3097.

I wasn't able to test it since the table wasn't populated with any companies when I tried to check locally. Not sure if I missed a step in setup.

In any case, since employer.commitment_categories appears to be an array, I believe this change should be all that's required.

patrickvinograd commented 7 years ago

LGTM. I think we have enough data in staging that I can verify this there before we go to production.