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

new alcatel_sros tmpl, 'show service sdp' #886

Merged
merged 6 commits into from
Mar 11, 2021
Merged

new alcatel_sros tmpl, 'show service sdp' #886

merged 6 commits into from
Mar 11, 2021

Conversation

h4ndzdatm0ld
Copy link
Contributor

  • New Template Pull Request

alcatel_sros_show_service_sdp,
alcatel_sros,
'show service sdp'

Additional template submission

Copy link
Contributor

@jmcgill298 jmcgill298 left a comment

Choose a reason for hiding this comment

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

Thanks @h4ndzdatm0ld

Matching Legends can be difficult, I believe my suggestions should work and avoid that section

templates/alcatel_sros_show_service_sdp.textfsm Outdated Show resolved Hide resolved
templates/alcatel_sros_show_service_sdp.textfsm Outdated Show resolved Hide resolved
templates/alcatel_sros_show_service_sdp.textfsm Outdated Show resolved Hide resolved
@h4ndzdatm0ld
Copy link
Contributor Author

Thanks @jmcgill298 - Moving to the Done state and dealing with the legends is great. I will keep this in mind for the next one. I committed the changes, tested and ran against a device - looks good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants