Skip to content

Conversation

@yulongzhang
Copy link
Contributor

Originally we only supported downloading by ExtID. This change enables downloading by ID as well

Originally we only supported downloading by ExtID. This change enables downloading by ID as well
if useExtID {
resp, err = get(fmt.Sprintf("/bug?extid=%v&json=1", id), token)
} else {
resp, err = get(fmt.Sprintf("/bug?id=%v&json=1", id), token)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would just try to download one link, and if it produced an error try to download the other one. It would make it simpler for users, and the code simpler too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants