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

New db schema #120

Merged
merged 53 commits into from
Apr 24, 2024
Merged

New db schema #120

merged 53 commits into from
Apr 24, 2024

Conversation

creme332
Copy link
Member

@creme332 creme332 commented Apr 24, 2024 •
edited
Loading

Checklist

Confirm you have completed the following actions prior to submitting this PR.

  • There is an existing issue report for this PR.
  • This is not a duplicate of an existing pull request.
  • My pull request is not resolving multiple issues at once.
  • I have created a feature branch.
  • My commit messages are detailed.
  • All of my commits are relevant to the issue.
  • composer test does not throw any error.

Issue Resolution

Tell us which issue this PR fixes.

This pull request Fixes #112

Fixes #56

Additional Information

If needed, provide additional information about your pull request.

creme332 added 30 commits April 20, 2024 12:48
@creme332
use datetime for review date
9dc4641
@creme332
create new version v1 of erd
d595b9f
@creme332
add background color to v1
46ed961
@creme332
add total participation: comment-writes, review-writes
8636784
@creme332
create a store model
1704945
@creme332
create Location model
7ee6bc0
@creme332
implement new version v1 of erd
e3b6f9a
@creme332
- replace latitude and longitude with a coordinate of type point
7187b0b 
- add sample data for store and store_product
@creme332
update diagrams based on v1 schema
0ccf800
@creme332
create comment model
1730ec7
@creme332
rename user_id to client_id in order table
410a970
@creme332
refactor getByID, add phpdoc for save()
54706f2
@creme332
update attributes based on new db schema, update signature of constru…
3f507c4 
…ctor
@creme332
new constructor, complete save(), fix bug in toArray
bce3cba
@creme332
add toArray(), store only district_id
40c09dd
@creme332
refactor getByID
5674b86
@creme332
remove stock level, add created_date attribute
fc70aa8
@creme332
change return type of save() to bool
2ab73ce
@creme332
add stringToDate function
f321fcf
@creme332
- store unit price in order_product relationship
3d6dafd 
- make total_price a derived attribute of order
- not all comments reply to a parent comment
- all comments belong to a review
@creme332
handle case where table names are reserved sql keywords
3e71d6c
@creme332
create OrderProduct model
92127c9
@creme332
rework order model based on new db schema
3f9620c
@creme332
- add unit_price to order_product
6bcda66 
- remove total price from order
- set default order status to pending
@creme332
refactor
d2222e7
@creme332
fix bugs
3a0ff69
@creme332
make improvements to UI
aa9a7fd 
reposition filters to avoid cluttering page

remove top padding from main so that first row of products is visible
@creme332
make getFullName static
b6be5f6
@creme332
make getNestedComments operational
bcb41ea
@creme332
no need to call function to get full name
d4ccb84
creme332 added 19 commits April 23, 2024 14:11
@creme332
fix bug for review_id in toArray
995093b
@creme332
make nested comments work
29c3b89
@creme332
remove unused import
67e03a8
@creme332
make nesting indentation dynamic
a894542
@creme332
fix date bug in getByID
8006f17
@creme332
handle case where modal components (close button) are null
4019508
@creme332
rename file, init comment form modal
251b263
@creme332
display comment form when reply button is clicked
a2ebb32
@creme332
handle comment submission
32a2818
@creme332
init modals after dom has loaded
4240948
@creme332
show error messages in comment form
1ffc7b5
@creme332
add getFormattedAddress
fc29bbb
@creme332
display store options on shopping cart page
da7fa1f
@creme332
add getAll, rename Id to ID
6b787d7
@creme332
- remove client data
ed0f00e 
- reset auto inc index for order, review, user
- set text data type to varchar(2000)
@creme332
update outdated diagrams
99b5b4d
@creme332
specify charset in dsn
3cc9260
@creme332
set default charset of comment to utf8mb4 to fix error when comment c…
9d83a01 
…ontains emojis
@creme332
rewrite part of FDs for client
e7d3583
This was linked to issues Apr 24, 2024
Implement nested comments on product page #112
Closed
Improve database schema #56
Closed
@creme332 creme332 merged commit 1af9233 into main Apr 24, 2024
0 of 2 checks passed
@creme332 creme332 deleted the new-db-schema branch April 24, 2024 07:46
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.

Implement nested comments on product page Improve database schema
1 participant
@creme332