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.

Sign up for GitHub

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

Jump to bottom

Better api error handling #62

Closed
wants to merge 4 commits into from
Closed

Better api error handling #62

wants to merge 4 commits into from

Commits on Feb 20, 2024

  1. Introduce FoojayService and rewrite FoojayApi 

    Signed-off-by: StefMa <StefMaDev@outlook.com>
    @StefMa
    StefMa committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    244f57a View commit details
    Browse the repository at this point in the history

  2. Return Result instead of throwing an error on api error 

    Signed-off-by: StefMa <StefMaDev@outlook.com>
    @StefMa
    StefMa committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    1031b9a View commit details
    Browse the repository at this point in the history

  3. Do not throw in error case 

    Signed-off-by: StefMa <StefMaDev@outlook.com>
    @StefMa
    StefMa committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    a1a6284 View commit details
    Browse the repository at this point in the history

  4. Fix test 

    Signed-off-by: StefMa <StefMaDev@outlook.com>
    @StefMa
    StefMa committed Feb 20, 2024
    Configuration menu
    Copy the full SHA
    2555271 View commit details
    Browse the repository at this point in the history