Track document downloads#2776
Conversation
susilnem
left a comment
susilnem
left a comment
There was a problem hiding this comment.
Overall, it looks good to me. Just some queries.
There was a problem hiding this comment.
Are we planning to run this Script manually?
There was a problem hiding this comment.
No, a cronjob setting will run it in a recurring time.
| choices=DocumentType.choices, | ||
| ) | ||
| # PK of the source record – nullable for purely external documents | ||
| object_id = models.PositiveIntegerField( |
There was a problem hiding this comment.
How do we check/verify this field?
There was a problem hiding this comment.
Not clear what you ask.
There was a problem hiding this comment.
Just a simple check to see whether the object_id exists in our system or not.
There was a problem hiding this comment.
This object_id can point to different systems. So it can be a duplicate one. Not clear yet the existence check.
|
Thanksl, @susilnem , for the thorougful review and comments! |
szabozoltan69
force-pushed
the
feature/doc-download-tracking
branch
from
1a5047e to
f295f69
Compare
szabozoltan69
force-pushed
the
feature/doc-download-tracking
branch
2 times, most recently
from
5757210 to
5fdcda1
Compare
|
@copilot resolve the merge conflicts in this pull request |
Done. Merged |
szabozoltan69
force-pushed
the
feature/doc-download-tracking
branch
from
e159222 to
7ae9d8d
Compare
szabozoltan69
force-pushed
the
feature/doc-download-tracking
branch
from
7ae9d8d to
c9da6b0
Compare
Refers to #2065 and to IFRCGo/go-web-app#750