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

Improve API Digestion stability and include discrepancies visualization #197

Merged
merged 19 commits into from
Oct 29, 2023

Conversation

KoolTheba
Copy link
Collaborator

@KoolTheba KoolTheba commented Oct 28, 2023

Main Changes

  • Add support for data inconsistency with the checks in the API reponse while comparing between commits. This normalize the data and make it consistent for the comparison.
  • Add alphabetical order for the checks
  • Add discrepacies in the checks. This new feature will show the checks that are not present in the other commit. This is useful when comparing between commits. This information is shown in the UI
  • Formatting, linting and minor code changes are done to improve the code legibility and maintainability
  • Added unit tests for the new features and the refactored code
  • Added discrepancies to the documentation

Discrepancies preview

Captura de pantalla 2023-10-29 a las 16 22 58

Context

Changelog

@UlisesGascon UlisesGascon changed the title WIP feat: improve mutable api data digestion Improve API Digestion stability and include discrepancies visualization Oct 28, 2023
UlisesGascon
UlisesGascon previously approved these changes Oct 28, 2023
@UlisesGascon UlisesGascon marked this pull request as ready for review October 28, 2023 22:09
Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the UI improvements in the last commits!

@KoolTheba KoolTheba merged commit d88af7c into main Oct 29, 2023
4 checks passed
@KoolTheba KoolTheba deleted the feat/improve-mutable-payloads branch October 29, 2023 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mixing results
2 participants