Fix for 7.1:
local link = GetQuestLink(index)
local link = GetQuestLink(id)
Reason: Blizzard changed the GetQuestLink API to use the quest's name rather than its index in your list of quests to generate a link.
Hopefully the author can make this change as well.