djangoflow / django-df-chat

Djangoflow in-app chat server using django-channels
MIT License
0 stars 16 forks source link

Suggestion: rename UserChat for verbosity #30

Closed dokterbob closed 1 year ago

dokterbob commented 1 year ago

Docstring for UserChat reads: This model stores attributes that should be visible across all rooms for a user.

The current name is suggestive to something more akin to a chat session whereas, quite literally, it's chat-related attributes of a user. Given that the model is already namespaced to df_chat, I would suggest renaming to UserAttributes (and use import as in external contexts) or UserState (as that's how it's currently used).

I think a change like this will significantly increase comprehension of the code and data structures.