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

A couple of typos spotted for the same word group in the documentation #10179

Closed
ibrahimtok opened this issue Aug 28, 2022 · 2 comments · Fixed by #10180
Closed

A couple of typos spotted for the same word group in the documentation #10179

ibrahimtok opened this issue Aug 28, 2022 · 2 comments · Fixed by #10180
Assignees
Labels
status: accepted This issue has been accepted for implementation type: documentation A change or addition to the documentation

Comments

@ibrahimtok
Copy link
Contributor

Change Type

Cleanup (formatting, typos, etc.)

Area

Other

Proposed Changes

typos spotted are on the below url;
https://github.com/netbox-community/netbox/blob/develop/docs/plugins/development/filtersets.md

# views.py
from netbox.views.generic import ObjectListView
-from .filtersets import MyModelFitlerSet
+from .filtersets import MyModelFilterSet
from .models import MyModel

class MyModelListView(ObjectListView):
    queryset = MyModel.objects.all()
-    filterset = MyModelFitlerSet
+    filterset = MyModelFilterSet

let's make netbox perfect in every aspect

@ibrahimtok ibrahimtok added the type: documentation A change or addition to the documentation label Aug 28, 2022
@DanSheps DanSheps added the status: accepted This issue has been accepted for implementation label Aug 29, 2022
@jeremystretch
Copy link
Member

I really need to set an auto-correct macro in my IDE for "fitler."

@DanSheps
Copy link
Member

Sounds like my common one...

serfver

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: accepted This issue has been accepted for implementation type: documentation A change or addition to the documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants