Skip to content
This repository has been archived by the owner on Jul 11, 2022. It is now read-only.

Remove mandatory variable Definitions and allow access token in header #7

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

salujaharkirat
Copy link
Collaborator

@salujaharkirat salujaharkirat commented Jun 14, 2018

For queries which don't require variables the code was breaking as _context.variableDefinitions would be null and upon reading variableDefinitions would throw error.

Adding following checks:-

  1. Reading variableDefinitions only if _context.variableDefinitions is present.
  2. If variables is set then assigning to final query else assigning null

Auth token changes:-

  1. allowing option to pass auth token in the api calls.

@salujaharkirat salujaharkirat changed the title Remove mandatory variable Definitions Remove mandatory variable Definitions and allow access token in header Jun 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant