Ideally we shouldn't be printing urls using the `print` function. We should use the `logging` module instead. Here's a guide! https://docs.python.org/3/howto/logging.html
I have also specified the payload type that the backend will recieve in that document.
In this case, the payload looks something like this: {"url": "http://example.com", "name": "An example link"}
Take a look at the diagram in the readme for the correct route :)
https://git.uwaterloo.ca/csc/linktree#architecture
I have also specified the payload type that the backend will recieve in that document.
In this case, the payload looks something like this: `{"url": "http://example.com", "name": "An example link"}`
changed this line in [version 2 of the diff](/csc/linktree/-/merge_requests/9/diffs?diff_id=51663&start_sha=04f85228c6d304a6aa5a9911f2814c6f977ad1f7#cb368897cc5bf4934e15bc7a3c0b7d3b69446875_33_32)
changed this line in [version 2 of the diff](/csc/linktree/-/merge_requests/9/diffs?diff_id=51663&start_sha=04f85228c6d304a6aa5a9911f2814c6f977ad1f7#cb368897cc5bf4934e15bc7a3c0b7d3b69446875_30_30)
changed this line in [version 2 of the diff](/csc/linktree/-/merge_requests/9/diffs?diff_id=51663&start_sha=04f85228c6d304a6aa5a9911f2814c6f977ad1f7#cb368897cc5bf4934e15bc7a3c0b7d3b69446875_44_45)
changed this line in [version 3 of the diff](/csc/linktree/-/merge_requests/9/diffs?diff_id=52026&start_sha=b614fdb93692b5cb2111d3bb8bde438aa7651366#cb368897cc5bf4934e15bc7a3c0b7d3b69446875_2_2)
added 1 commit
<ul><li>cea734ef - first draft for get_links()</li></ul>
[Compare with previous version](/csc/linktree/-/merge_requests/9/diffs?diff_id=52026&start_sha=b614fdb93692b5cb2111d3bb8bde438aa7651366)
added POST method to update click count in database per click
Ideally we shouldn't be printing urls using the
printfunction. We should use theloggingmodule instead. Here's a guide! https://docs.python.org/3/howto/logging.htmlTake a look at the diagram in the readme for the correct route :)
https://git.uwaterloo.ca/csc/linktree#architecture
I have also specified the payload type that the backend will recieve in that document.
In this case, the payload looks something like this:
{"url": "http://example.com", "name": "An example link"}You can remove this import, there's already one below of the same package (on line 5).
You can remove this comment, and then add a one line comment just above the function describing it (doesn't have to be too detailed).
Left a few more comments, but looks good overall!
requested review from @n3parikh
changed this line in version 2 of the diff
changed this line in version 2 of the diff
changed this line in version 2 of the diff
added 1 commit
Compare with previous version
changed this line in version 3 of the diff
added 1 commit
Compare with previous version
added 2 commits
Compare with previous version
added 5 commits
mainCompare with previous version
approved this merge request
Reviewers