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

Replace underscores in plugin app name with dashes #397

Merged
merged 2 commits into from
Jul 30, 2021

Conversation

dvaccarosenna
Copy link
Contributor

Ran into this issue with my own plugins, where the plugin name is "-" separated in the URL. This fixes issue 332.

@zachmoody
Copy link
Contributor

Thanks! Looks good, happy to merge this, but could you make the linter happy and replace those single-quotes in the test?

@dvaccarosenna
Copy link
Contributor Author

No problem, updated the quotes in the test.

@zachmoody zachmoody merged commit cf9a4c8 into netbox-community:master Jul 30, 2021
@dvaccarosenna dvaccarosenna deleted the issue/332 branch August 2, 2021 08:57
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.

Dash-separated Plugins Endpoint name support
2 participants