fix: updated the full-status api to use swagger spec for counts #343

VinayJariya · 2023-06-12T11:45:36Z

Resolves Issue #300 .
Updated swagger category
Updated endpoint to use data from swaggerSpec (same as astro)

what-the-diff · 2023-06-12T11:46:03Z

PR Summary

Addition of new tags in API documentation
New tags have been added to the swagger.json file, improving the API documentation.
Dynamic calculation of app status endpoint values
The app status endpoint now fetches total categories, endpoints, and version number directly from the swaggerSpec object, removing the need for hardcoded values.
Removal of outdated import
An obsolete import for an old Swagger file has been removed, as the current project utilizes a custom-generated swaggerSpec from the utils/swagger module.

Signed-off-by: Aaron Rackley (EverydayTinkerer) <aaron.rackley9@gmail.com>

ghost · 2023-06-13T13:03:43Z

👇 Click on the image for a new way to code review

ageddesi · 2023-06-13T13:04:49Z

Hey @VinayJariya Thank you for your contribution to the platform

VinayJariya · 2023-06-14T06:23:05Z

Hey @VinayJariya Thank you for your contribution to the platform

Happy to contribute 😄

Vinay Jariya added 2 commits June 12, 2023 17:14

fix: updated the swagger for category count

52a46e3

fix: updated the full-status api to use swagger spec for counts

efd4185

VinayJariya changed the base branch from main to dev June 12, 2023 11:45

ageddesi previously approved these changes Jun 13, 2023

View reviewed changes

fix: codacy fixes

eb75fb1

Signed-off-by: Aaron Rackley (EverydayTinkerer) <aaron.rackley9@gmail.com>

ageddesi dismissed their stale review via eb75fb1 June 13, 2023 13:02

ageddesi approved these changes Jun 13, 2023

View reviewed changes

ageddesi merged commit 8b2d96d into ageddesi:dev Jun 13, 2023
2 checks passed