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

Merge: dev -> main #9

Merged
merged 21 commits into from
Sep 27, 2024
Merged

Merge: dev -> main #9

merged 21 commits into from
Sep 27, 2024

Conversation

hamdi3
Copy link
Owner

@hamdi3 hamdi3 commented Sep 27, 2024

No description provided.

hamdi3 and others added 21 commits September 21, 2024 11:34
Fixed imports for PKCS1 to use it a signature instead of a cipher
Integrated Flask into it and implemented the routes
Added routes tests
Added Flask to req.
Added docker support
Added simple transaction validators
Updated the tests for transaction validators
Updated the route tests to cover the cases of walets not being found
Updated tests to improve the coverage
Removed the transaction class and added the blockchain class
Updated the routes based on the new blockchain class & added more routes
Updated the tests
Removed the version from the docker-compose.yaml file since it's obsolete
Updated the requirements.txt file
Code fixes
- Added OpenAI(Swagger) to document the api.
- Updated the routes to use JSON instead of forms (for Consistency, flexibility, scalability).
Updated the requirements.txt
- Updated the routes tests to accept JSON instead of forms
- Increased coverage
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines Coverage Δ
src/app/__init__.py 100.00% <100.00%> (ø)
src/app/routes.py 100.00% <100.00%> (+14.81%) ⬆️
tests/test_routes.py 99.63% <100.00%> (+0.37%) ⬆️

@hamdi3 hamdi3 merged commit 95db182 into main Sep 27, 2024
1 check passed
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.

2 participants