Skip to content
This repository has been archived by the owner on Apr 12, 2022. It is now read-only.

Display list of Destinations on record/edit page #2641

Merged
merged 1 commit into from
Dec 6, 2021
Merged

Display list of Destinations on record/edit page #2641

merged 1 commit into from
Dec 6, 2021

Conversation

evantahler
Copy link
Member

@evantahler evantahler commented Dec 6, 2021 •
edited

Screen Shot 2021-12-06 at 10 55 37 AM

Checklists

Development

  • Application changes have been tested appropriately

Impact

  • Code follows company security practices and guidelines
  • Security impact of change has been considered
  • Performance impact of change has been considered
  • Possible migration needs considered (model migrations, config migrations, etc.)

Please explain any security, performance, migration, or other impacts if relevant:

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached where applicable.
  • Relevant tags have been added to the PR (bug, enhancement, internal, etc.)

@evantahler evantahler added the enhancement New feature or request label Dec 6, 2021
@evantahler evantahler marked this pull request as ready for review December 6, 2021 18:57
edmundito
edmundito approved these changes Dec 6, 2021
View reviewed changes
Copy link
Contributor

@edmundito edmundito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@evantahler evantahler merged commit 475d219 into main Dec 6, 2021
@evantahler evantahler deleted the 180546192-show-record-destinations branch December 6, 2021 20:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@edmundito edmundito edmundito approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@evantahler @edmundito