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

Support custom error handlers #17

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

Support custom error handlers #17

wants to merge 4 commits into from

Conversation

mmartell37
Copy link
Contributor

This is a work in progress based on #16, partially addressing #13.

The desire is to allow custom error handling as defined by the caller and to provide a few basic error handers.

An example handler class is provided for testing as well as a very basic default which simply displays the error message.

Currently, caller-specified error handlers may be ignored at initial render. See below.

To do:

  • Provide additional, possibly configurable default error handler options such as one that raises the thrown exception rather than simply displaying it and moving on.
  • Determine the initialization order such that caller-defined error handlers will exist before Consult.render! is executed.

sixfeetover and others added 3 commits September 26, 2018 11:44
This will concatenate the contents of each file together to form a single template, which will then be rendered once.
This will concatenate the contents of each Consul key together to form a single template, which will then be rendered once.

This will also concatenate the contents of single or multiple files if present.

Consul keys are concatenated after files.
@mmartell37 mmartell37 changed the title Support custom error handling Support custom error handlers Oct 9, 2018
@mmartell37 mmartell37 requested a review from sixfeetover October 9, 2018 13:26
@sixfeetover sixfeetover changed the base branch from safe-rerender to master October 10, 2018 16:51
@sixfeetover
Copy link
Member

This was closed accidentally due to deleting the base branch.

@sixfeetover sixfeetover reopened this Oct 10, 2018
@sixfeetover
Copy link
Member

This needs to be rebased on the latest master due to our merging the other branches.

@aharpervc aharpervc marked this pull request as draft March 16, 2021 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants