nautobot-app-circuit-maintenance icon indicating copy to clipboard operation
nautobot-app-circuit-maintenance copied to clipboard

When Nautobot Notes Are Implemented, Changes May Be Needed

Open jvanderaa opened this issue 2 years ago • 3 comments

Environment

  • Current

Proposed Functionality

Noticed that we have "Notes" as part of the current model. It looks like a global implementation may be upcoming. If this is implemented in Nautobot, we should remove/change the naming to avoid confusion - https://github.com/nautobot/nautobot/pull/1984

Use Case

Prevent confusion of models

jvanderaa avatar Jul 03 '22 13:07 jvanderaa

Totally, this was the first thought I got when I saw the new notes feature for Nautobot 1.4. I think that there is too much duplication, so I would tend to remove the plugin one.

chadell avatar Jul 04 '22 04:07 chadell

Agreed, will investigate this soon as 1.4 was released today on 16 Aug. `22

scetron avatar Aug 16 '22 16:08 scetron

This also causes some warnings during startup of Nautobot 2.0 with this app installed:

 ?: (drf_spectacular.W001) /opt/nautobot/.local/lib/python3.11/site-packages/nautobot_circuit_maintenance/api/serializers.py: Warning [MaintenanceNoteTaskView > NoteSerializer]: Encountered 2 components with identical names "Note" and different classes <class 'nautobot_circuit_maintenance.api.serializers.NoteSerializer'> and <class 'nautobot.extras.api.serializers.NoteSerializer'>. This will very likely result in an incorrect schema. Try renaming one.
 ?: (drf_spectacular.W001) /opt/nautobot/.local/lib/python3.11/site-packages/nautobot_circuit_maintenance/api/serializers.py: Warning [MaintenanceNoteTaskView > NoteSerializer]: Encountered 2 components with identical names "NoteRequest" and different classes <class 'nautobot_circuit_maintenance.api.serializers.NoteSerializer'> and <class 'nautobot.extras.api.serializers.NoteSerializer'>. This will very likely result in an incorrect schema. Try renaming one.
 ?: (drf_spectacular.W001) /opt/nautobot/.local/lib/python3.11/site-packages/nautobot_circuit_maintenance/api/serializers.py: Warning [MaintenanceNoteTaskView > NoteSerializer]: Encountered 2 components with identical names "PatchedNoteRequest" and different classes <class 'nautobot_circuit_maintenance.api.serializers.NoteSerializer'> and <class 'nautobot.extras.api.serializers.NoteSerializer'>. This will very likely result in an incorrect schema. Try renaming one.
 ?: (drf_spectacular.W001) /opt/nautobot/.local/lib/python3.11/site-packages/nautobot_circuit_maintenance/api/views.py: Warning [MaintenanceNoteTaskView]: Encountered 2 components with identical names "PaginatedNoteList" and different classes <class 'nautobot_circuit_maintenance.api.serializers.NoteSerializer'> and <class 'nautobot.extras.api.serializers.NoteSerializer'>. This will very likely result in an incorrect schema. Try renaming one.

glennmatthews avatar Nov 10 '23 21:11 glennmatthews