Avoid having extra ? at the end of request url when queryItems are empty

[tranvutuan]

Summary

Avoid having extra ? at the end of request url when queryItems are empty

Details

Currently GraphQLGETTransformer is used to create query items for GET request endpoint. If property body of this struct is empty ( does not have value for a given key) such as:

key : "variables"
value : nil

which leads to queryItems is empty. By doing components.queryItems = queryItems, then the url ends up to have extra ? at the end like following: https://this_is_sample/abcd? instead of https://this_is_sample/abcd

Fix

Only assign queryItems to URLComponent when it is not empty

[apollo-cla]

@tranvutuan: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

[designatednerd]

@tranvutuan I can't merge this until you sign the CLA, but this looks good. Mind signing it?

[tranvutuan]

@tranvutuan I can't merge this until you sign the CLA, but this looks good. Mind signing it?

@designatednerd I'm still waiting for a permission to sign the CLA from our end. I'll do right away when get approval

[designatednerd]

Cool, thanks for the heads up.

[tranvutuan]

Cool, thanks for the heads up.

Got a green light from our end and I have just signed the CLA.

[designatednerd]

Woo hoo, thank you!