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

Move error handling for delete proxy backend method into callback #1688

Closed
bajiat opened this issue Oct 5, 2016 · 1 comment
Closed

Move error handling for delete proxy backend method into callback #1688

bajiat opened this issue Oct 5, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@bajiat
Copy link
Contributor

bajiat commented Oct 5, 2016

When adding the rate limiting field, we added error handling using try catch. Since this is related to a Meteor method, error handling should happen inside the callback. Move error handling for delete proxy backend method into the Meteor method callback.

Example

// Calling a Meteor method with error handling
Meteor.call('methodName', function (error, result) {
  if (error) {
    // Do something with the error
  } else {
    // Do something if no error
  }
});
@bajiat bajiat added this to the Sprint 32 milestone Oct 5, 2016
@bajiat
Copy link
Contributor Author

bajiat commented Oct 5, 2016

@brylie Provide a link to the code in question.

@brylie brylie changed the title Move error handling for rate limit field Move error handling for delete proxy backend method into callback Oct 5, 2016
@jykae jykae added in progress and removed ready labels Oct 5, 2016
@brylie brylie removed the in progress label Oct 5, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants