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

Feature/add multiple FoS at once #466

Merged
merged 11 commits into from
Sep 14, 2020

Conversation

tobias-wiese
Copy link
Contributor

@tobias-wiese tobias-wiese commented Sep 14, 2020

Description

Implements issue #464 (delete if no issue)

Reason for this PR

Changes in this PR

  • change required arguments for updateMatriculation a list of SubjectMatriculatio and apapt the API calls

Type of change (remove all that don't apply)

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Could not be tested yet as MatriculationManagement is not available

Checklist: (remove all that don't apply)

  • I added corresponding unit tests (if API code)
  • I have performed a self-review of my own code
  • My changes generate no new linting warnings or console warnings
  • I have updated the CHANGELOG.md to include any changes made in this PR (add WIP to the top, if there is none already)

Notes:

@tobias-wiese tobias-wiese added enhancement New feature or request vue labels Sep 14, 2020
src/api/MatriculationManagement.ts Outdated Show resolved Hide resolved
@tobias-wiese tobias-wiese merged commit 0ce679f into develop Sep 14, 2020
@tobias-wiese tobias-wiese deleted the feature/add_multiple_FoS_at_once branch September 14, 2020 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request vue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants