Skip to content

#2313 added a :created status on successful save #2348

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.

Sign up for GitHub

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

Jump to bottom
Merged
rlgreen91 merged 7 commits into from
Sep 5, 2018
Merged

#2313 added a :created status on successful save #2348

rlgreen91 merged 7 commits into from
Sep 5, 2018

Conversation

@danuluma
Copy link
Collaborator

@danuluma danuluma commented Sep 3, 2018

added status: :created to be returned when an agency is successfully created.

In your PR did you:

  • Include a description of the changes?
  • Mention the issue the PR addresses?
  • Include screenshots of any changes to the UI?
  • Isolate any changes to gems (meaning that any new, updated, or removed gems and resulting code changes should be in their own PR)?
  • Add and/or update specs for your code?

@rlgreen91
Copy link
Contributor

rlgreen91 commented Sep 3, 2018

@danuluma Please address the CodeClimate issue.

@rlgreen91 rlgreen91 merged commit edf9110 into master Sep 5, 2018
@rlgreen91 rlgreen91 deleted the agency-controller-2313 branch September 5, 2018 03:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@danuluma @rlgreen91