Skip to content

Release/version 0.2.0 #27

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
Open
wants to merge 99 commits into
base: main
Choose a base branch
from
Open

Release/version 0.2.0 #27

wants to merge 99 commits into from

Conversation

@Lakorthus
Copy link
Contributor

@Lakorthus Lakorthus commented Jul 19, 2023

What

How to review

Follow Readme file.

Who can review

@nshumoogum

Lakorthus and others added 30 commits June 20, 2023 12:53
@Lakorthus @piratejas
Create Hello world endpoint
2089170 
Co-authored-by: Pira Tejasakulsin <[email protected]>
@piratejas
tested and updated readme instructions for running app
1571f0a
@Lakorthus @piratejas
test: test_hello_world controller function.
ea1d499 
Co-authored-by: Pira Tejasakulsin <[email protected]>
@Lakorthus
test: routes on hello_world
0af031b
@Lakorthus
Merge pull request #2 from methods/feat_hello_world
b0d5802 
Bids API: Create Hello world endpoint
@piratejas @Lakorthus
feat: added and manually tested post route for bid
63c02ba 
Co-authored-by: Julio Velezmoro <[email protected]>
@Lakorthus @piratejas
feat: saving data in memory and separating logic
ac87d4e 
Co-authored-by: Pira Tejasakulsin <[email protected]>
@piratejas @Lakorthus
wip: add phase info to success/failed
d1f230c 
Co-authored-by: Julio Velezmoro <[email protected]>
@piratejas @Lakorthus
feat: field validation, set status enum
53d3913 
Co-authored-by: Julio Velezmoro <[email protected]>
@piratejas
test: add post bid route tests
3d0dc1c
@piratejas
Merge pull request #3 from methods/feat/create-new-bid
7641b4b 
Feat/create new bid
@piratejas
fix: renamed files and folders for readability
2d6d8cf
@piratejas
Merge pull request #4 from methods/fix/filenames-improvement
65901b1 
fix: renamed files and folders for readability
@piratejas
fix: changed set_status
027cc43
@piratejas
wip: swagger spec config
817cd3d
@nshumoogum
docs: Update contribution guidelines to incl. release and tagging
cc3763c
@piratejas
fix: installed flask_swagger_ui; created swagger_config.yml
34c1a45
@Lakorthus
chore: flask-ui added to the requirements
44145f6
@Lakorthus
Merge branch 'feat/swagger-spec' into fix/set-status-enum
4a35b21
@Lakorthus
Merge pull request #7 from methods/fix/set-status-enum
03520b5 
fix: changed set_status
@piratejas
fix: swagger config structure
2d2d28f
@piratejas @Lakorthus
refactor: swagger config components and refs
5f10e67 
Co-authored-by: Julio Velezmoro <[email protected]>
@piratejas
feat: completed swagger spec for post bid endpoint
cd2bc74
@piratejas
Merge pull request #8 from methods/feat/swagger-spec
0e0677e 
Feat/swagger spec
@piratejas @Lakorthus
refactor: success and failed phase info
9249e09 
Co-authored-by: Julio Velezmoro <[email protected]>
@piratejas
refactor: added helper function and logic for feedback
6d1fdbc
@piratejas
Merge pull request #9 from methods/refactor/post-bid-endpoint
3ac5bd2 
Refactor/post bid endpoint
@piratejas
refactor: added marshmallow schemas; renamed classes and fixed import…
8d7aaac 
…s; serialised bid_doc using schema
@piratejas
refactor: handling ValidationError with marshmallow and outputting cu…
1b48bc2 
…stom error response
@piratejas
refactor: added test request bodies to check validation
ed395ce
piratejas and others added 29 commits July 12, 2023 12:24
@piratejas
feat: added query to get_all to filter by status not equals to deleted
06adef2
@Lakorthus
refactor: testing without magicMock
49ad68a
@piratejas
fix: updated soft delete method from PUT to DELETE
b5add45
@piratejas
feat: added swag spec and endpoint for update bid route; changed bid …
1a7ba79 
…request schema and bid model to allow reusability
@piratejas
refactor: updated examples in swagger spec
c841aeb
@piratejas
Merge pull request #20 from methods/feat/filtering-out-deleted
6ceb406 
feat: added query to get_all to filter by status not equals to deleted
@Lakorthus
Merge pull request #21 from methods/refactor/test-magicMock
222f5a2 
refactor: testing without magicMock
@piratejas
Merge branch 'develop' into feat/update-bid-by-id
700d658
@piratejas
Merge pull request #22 from methods/feat/update-bid-by-id
8b26e55 
Feat/update bid by
@piratejas
test: update bid by id tests; refactored to add helper functions for …
b3b574b 
…input validation
@piratejas
refactor: removed unused imports
6f2f6a2
@piratejas
refactor: cleaned up controller
f9ae9e9
@piratejas
refactor: changed find and replace to update one; wip: adding last_up…
1238a99 
…dated timestamp in correct format
@piratejas
test: updating tests after refactor
1c986c2
@piratejas
test: updated tests after refactoring
dcd3f6a
@piratejas
Merge pull request #23 from methods/test/update-bid-by-id
386a13b 
Test/update bid by
@Lakorthus
feat: Makefile
a9f169c
@Lakorthus
refactor: README and req.txt updated with Makefile
f573dcb
@Lakorthus
feat: vulnerability check using safety
99d5a63
@piratejas
refactor: added custom error message to path param validation
8859be2
@Lakorthus
feat: commit script Makefile
8329de6
@Lakorthus
feat: commit message to gitignore
b6f3e0e
@Lakorthus
refactor: makefile check
738434a
@Lakorthus
test: testing commit msg
d642f7a
@Lakorthus
test: test commit msg
eea3c3b
@Lakorthus
test:
9a4b05c
@Lakorthus
Merge pull request #25 from methods/feat/gmakefile
b648ac0 
feat: Makefile
@piratejas
refactor: updated swagger to include additional error responses
cc28971
@piratejas
Merge pull request #26 from methods/refactor/error-handling
b7a95d4 
Refactor/error handling
@Lakorthus Lakorthus requested a review from nshumoogum July 19, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nshumoogum nshumoogum Awaiting requested review from nshumoogum

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Lakorthus @piratejas @nshumoogum