Skip to content

Create axl_update_DN_Description_Alerting_CallerID#15

Open
seenamoshiri wants to merge 3 commits into
CiscoDevNet:masterfrom
seenamoshiri:patch-1
Open

Create axl_update_DN_Description_Alerting_CallerID#15
seenamoshiri wants to merge 3 commits into
CiscoDevNet:masterfrom
seenamoshiri:patch-1

Conversation

@seenamoshiri

Copy link
Copy Markdown

No description provided.

@dstaudt

dstaudt commented Mar 22, 2022

Copy link
Copy Markdown
Contributor

Thanks for the contribution :)

As with the other bulk sample:

  • Could you modify the sample in order to create/(do main operation)/delete test data to avoid dependency on pre-existing data and/or accidentlly modifying existing data? Here, perhaps also include a sample .csv file that operates on the test objects, e.g. in : ../sample_data/axl_update_DN_Description_Alerting_CallerID.csv
  • Perhaps add some minimal error handling to avoid a bunch of cascading errors if an operation fails unexpectedly?

@seenamoshiri

Copy link
Copy Markdown
Author

Hi :)

  • Done
  • Done

If there are any other problems or ideas to help make this better I'm ready to work on it! :)

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