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.

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

update job hook to use slug in urls #2132

Merged
merged 1 commit into from
Aug 2, 2022
Merged

update job hook to use slug in urls #2132

merged 1 commit into from
Aug 2, 2022

Conversation

gsnider2195
Copy link
Contributor

@gsnider2195 gsnider2195 commented Aug 2, 2022 •
edited
Loading

Closes: #DNE

What's Changed

  • Job hook model updated to use slug in url instead of pk

TODO

  • Explanation of Change(s)
  • Attached Screenshots, Payload Example
  • Unit, Integration Tests
  • Documentation Updates (when adding/changing features)
  • Example Plugin Updates (when adding/changing features)
  • Outline Remaining Work, Constraints from Design

@gsnider2195 gsnider2195 merged commit ac73834 into next Aug 2, 2022
@gsnider2195 gsnider2195 deleted the gas-job-hook-detail-fix branch August 2, 2022 15:50
gsnider2195 added a commit that referenced this pull request Aug 2, 2022
@gsnider2195
add release note for #2132
a34cc15
smk4664 added a commit to smk4664/nautobot that referenced this pull request Aug 4, 2022
@smk4664
Merge branch 'next' into NotesAPI
0149176 
* next:
  update job hook docs, move to docs/models/extras (nautobot#2133)
  add release note for nautobot#2132
  update job hook to use slug in urls (nautobot#2132)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@glennmatthews glennmatthews glennmatthews approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@gsnider2195 @glennmatthews