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.

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

Feat/item repository spec #956

Open
wants to merge 62 commits into
base: master
Choose a base branch
from
Open

Feat/item repository spec #956

wants to merge 62 commits into from

Conversation

b-shears
Copy link

@b-shears b-shears commented Jun 3, 2022

Bryan & Wes fixed merge conflicts and wrote update & delete tests & methods.

sunny-moore and others added 30 commits June 1, 2022 13:45
@sunny-moore
Test Merchant exists
ff3a3f5
@sunny-moore
Create Merchant class
031ebf6
@sunny-moore
Test Merchant Repository exist
8351500
@sunny-moore
Create Merchant Repository class
f613758
@sunny-moore
Test merch repo can return array of all known merchants
c6cc510
@sunny-moore
Test find_by_id returns all known merchants or nil
743f3a3
@sunny-moore
Create find_by_id method
68c559d
@wes30303
Merge pull request #1 from sunny-moore/feat/merchant-class
7c43c6f 
Feat/merchant class
@sunny-moore
Test Item exists
c756298
@sunny-moore
Create Item Class
d051f4c
@sunny-moore
Create Item Repository class
1350341
@gwen-marina
Merge pull request #2 from sunny-moore/feat/item-and-item-repository-…
2652bb0 
…class

Feat/item and item repository class
@gwen-marina
Test/added test for SalesEngine exists -GR
1b04f61
@gwen-marina
Feat/added code for initialize/self.from_csv method - GR
72f0482
@gwen-marina
Test/test for returning an array of all items -GR
47fae6d
@gwen-marina
Test/can return an array of all merchants - GR
973e89a
@b-shears
Merge pull request #3 from sunny-moore/feat/sales-engine
7d06865 
Feat/sales engine
@gwen-marina
Test/test for item_repository exists- GR
754ef3f
@gwen-marina
Fix/ fixed spacing and indents per issues found with Hound - GR
bacaa7f
@gwen-marina
Test/ added test for returning an array of all known items - GR
ac1d2ce
@b-shears
Merge pull request #4 from sunny-moore/feat/item-repository
e8c3897 
Feat/item repository
@wes30303
Test for merchant repository
1c05f83
@wes30303
Create methods for merchant rep
ee37181
@b-shears
Create find_all_by_price_in_range tests
8857c89
@gwen-marina
Test/can find by id test written -GR
8794882
@gwen-marina
Feat/wrote code for find_by_id method - GR
3e1a659
@gwen-marina
Test/added more robust test - GR
00df661
@gwen-marina
Test/added test to find item by name or return nil-GR
0f49961
@gwen-marina
Feat/added code for find_by_name method-GR
82d9324
@b-shears
Create find_all_by_in_range(range) method
6e84d40
gwen-marina and others added 30 commits June 1, 2022 22:01
@gwen-marina
Feat/wrote method for find all with description-GR
03b028f
@gwen-marina
Test/wrote test returns a unit price as a float -GR
9f441e2
@gwen-marina
Feat/wrote method for unit_price_to_dollars-GR
22820d4
@b-shears
Update find_all_by_price_in_range method
56f0dce
@b-shears
Update find_all_by_price_in_range test
4919fca
@b-shears
Update all variable to all_items
2cbddb8
@sunny-moore
Merge pull request #6 from sunny-moore/feat/item_repository-find-crea…
4f530f1 
…te-update

Feat/item repository find create update
@b-shears
Merge branch 'master' into feat/merchant_methods_added
3efd793
@b-shears
Merge pull request #5 from sunny-moore/feat/merchant_methods_added
9a42de1 
Feat/merchant methods added
@gwen-marina
Cleaned up empty space and indents that were messed up -GR
3c7cb05
@sunny-moore
Merge branch 'master' into feat/item-repository
263d1f6
@sunny-moore
Merge pull request #7 from sunny-moore/feat/item-repository
c8fc3c1 
Feat/item repository
@b-shears
Create find_all_by_merchant_id tests
80da734
@b-shears
Update find_all_by_merchant_id method
06e712f
@b-shears
Update find_all_by_merchant_id test
8813081
@b-shears
Update item_repository method
c2d399c
@b-shears
Update item_repository test
8d39787
@b-shears
Change attr_reader to attr_accessor
9742b85
@b-shears
Create update method test for item repository
7c247af
@b-shears
Update item_repository update test & method
0d24b7b
@gwen-marina
Test/added can find an item description by supplied price-Gr
e66e8fa
@gwen-marina
Feat/added code for find all by price - GR
a66d3fb
@gwen-marina
Fix/having issuses pushing up to main/trying to fix confilcts
1fb699a
@gwen-marina
Test/added test for find all by price
77d8343
@b-shears
Merge pull request #9 from sunny-moore/feat/item-repository
56ea97d 
Feat/item repository
@sunny-moore
Merge branch 'master' into feat/item_repository_spec
7ababdb
@b-shears
Fix merge conflicts & add delete & update method tests
e4c8ca3
@b-shears
Fix merge conflicts & add update & delete methods
3a624b1
@b-shears
Fix merge conflicts in lib/merchant file.
91592eb
@b-shears
Refactor find_all_by_price test
8145925
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@b-shears @sunny-moore @wes30303 @gwen-marina