paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-02 13:03:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

cache: Remove a now-unnecessary TODO.

Browse Source 
We solved the problem the TODO raised by using a different type
annotation syntax, and I'm not sure whether that refactor would
actually improve the code.
This commit is contained in:
Tim Abbott
2018-03-16 11:32:14 -07:00
parent 966ca7015f
commit 707af5ab56
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
zerver/lib/cache.py
View File

@@ -318,9 +318,6 @@ def user_profile_by_id_cache_key(user_profile_id: int) -> Text:
def user_profile_by_api_key_cache_key(api_key: Text) -> Text:
return "user_profile_by_api_key:%s" % (api_key,)
# TODO: Refactor these cache helpers into another file that can import
# models.py so that python v3 style type annotations can also work.
realm_user_dict_fields = [
'id', 'full_name', 'short_name', 'email',
'avatar_source', 'avatar_version', 'is_active',