Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Good job reusing the mainRequest function #6

Closed
Razpudding opened this issue May 27, 2017 · 0 comments
Closed

Good job reusing the mainRequest function #6

Razpudding opened this issue May 27, 2017 · 0 comments

Comments

@Razpudding
Copy link

Pretty smart to look at what the response look like and then to call a different function based on that.
Alternatively, you could pass two parameters to mainRequest. 1 being the targetPage and 2 being the callback function. That way you can tell mainRequest which function to pass the resulting data to. This is also described in the week 6 assignment.

gmamar added a commit that referenced this issue May 30, 2017
@gmamar gmamar closed this as completed in b709de5 May 30, 2017
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

No branches or pull requests

1 participant