Add links from DB to JSON function #12

Not sure if I did this correctly.

Looks good to me so far!

What's the format of the JSON? We want something like: [{'name': '...', 'url': ...}, ...]. I'm just a little confused because of the dict and zip call, but if it matches that, should be good to merge.

We should add some tests, but we can do that as part of a separate MR/later.

Here's the output if I print the resulting JSON:

[
    {
        "url": "http://csclub.uwaterloo.ca/",
        "name": "CS Club Website"
    },
    {
        "url": "https://www.instagram.com/uwcsclub/",
        "name": "Instagram"
    },
    {
        "url": "https://www.facebook.com/uw.computerscienceclub",
        "name": "Facebook"
    },
    {
        "url": "http://twitch.tv/uwcsclub",
        "name": "Twitch"
    },
    {
        "url": "http://bit.ly/uwcsclub-yt",
        "name": "YouTube"
    }
]

I just used the dict and zip call as a generalized way to match column names to column values.

Looks good to me!

approved this merge request

added 4 commits

817f5b53...6a3c29da - 3 commits from branch main
a0362b99 - Merge branch 'main' into william/regen-JSON

Compare with previous version

Add links from DB to JSON function #12

Reviewers