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

Adds block context on respond_to method #10

Merged
merged 1 commit into from
Oct 9, 2020

Conversation

dlinch
Copy link
Collaborator

@dlinch dlinch commented Oct 3, 2020

Address Issue 8
Updated ReadMe, including some markdown linting.

Update respond_to method to take a block whose return value is available within a route so regex isn't the only way to pass data through.

@dlinch
Copy link
Collaborator Author

dlinch commented Oct 3, 2020

@jejacks0n If you add the hacktoberfest topic to the repo then I get credit for hacktoberfest for this PR :)

@jejacks0n
Copy link
Owner

You should do that do I know what you mean. You’re a collaborator now.

@dlinch
Copy link
Collaborator Author

dlinch commented Oct 8, 2020

@jejacks0n Can I merge this bad boy then?

@jejacks0n
Copy link
Owner

jejacks0n commented Oct 8, 2020 via email

@dlinch dlinch merged commit 525f77d into jejacks0n:master Oct 9, 2020
@dlinch dlinch deleted the add-block-context-to-respond-to branch October 9, 2020 15:41
@mikepack
Copy link

mikepack commented Oct 9, 2020

giphy

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

Successfully merging this pull request may close these issues.

3 participants