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

[MINOR FEATURE]: Capture ApodiniError/@Throws in the APIDocument #7

Open
1 task done
Supereg opened this issue Dec 27, 2021 · 0 comments
Open
1 task done

[MINOR FEATURE]: Capture ApodiniError/@Throws in the APIDocument #7

Supereg opened this issue Dec 27, 2021 · 0 comments
Labels
enhancement New feature or request

Comments

@Supereg
Copy link
Member

Supereg commented Dec 27, 2021

Problem

As of now, the Migrator InterfaceExporter won't index the Apodini web service defined ApodiniErrors. Instead it uses a standard set of default ErrorCodes.

Solution

Parse @Throws declarations inside a Handler in the ApdoniMigratorInterfaceExporter and properly build the APIDocument. Additionally, changes in error codes should be captured by the MigrationGuide.

Additional context

Ideally, the RESTMigrator would be updated to adjust for the updated error code definitions. This issue does not require to come up with a meaningful way of handling error migrations.

Code of Conduct

  • I agree to follow this project's Code of Conduct and Contributing Guidelines
@Supereg Supereg added the enhancement New feature or request label Dec 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant