MountainTownTechnology
/
aural_isle
4
1
Fork 1
Code Issues 20 Pull Requests 3 Packages Projects 1 Releases Wiki Activity

Setup Initial Schema (sqlite) #10

Open
sundog wants to merge 4 commits from setup-initial-schema into main
pull from: setup-initial-schema
merge into: MountainTownTechnology:main
Conversation 2 Commits 4 Files Changed 8 +2320 -28
sundog commented 2023-11-01 10:18:55 -04:00
Owner
Copy Link

This PR introduces the initial database schema in the form of a SQLx migration. (See https://tms-dev-blog.com/rust-sqlx-basics-with-sqlite/ for a brief intro to SQLx).

Please review and comment. If there are no major objections over the next couple days then we'll go ahead and merge this as our starting point.

This PR introduces the initial database schema in the form of a SQLx migration. (See https://tms-dev-blog.com/rust-sqlx-basics-with-sqlite/ for a brief intro to SQLx). Please review and comment. If there are no major objections over the next couple days then we'll go ahead and merge this as our starting point.
sundog added 2 commits 2023-11-01 10:18:55 -04:00
dag commented 2023-11-02 08:54:01 -04:00
Collaborator
Copy Link

Do you think we might want to break out a PersonStreetAddress table to handle things like separate billing/shipping addresses, multiple shipping addresses, etc?

Do you think we might want to break out a PersonStreetAddress table to handle things like separate billing/shipping addresses, multiple shipping addresses, etc?
👍 1
sundog added 1 commit 2023-12-01 11:58:03 -04:00
sundog commented 2023-12-01 11:59:48 -04:00
Author
Owner
Copy Link

Do you think we might want to break out a PersonStreetAddress table to handle things like separate billing/shipping addresses, multiple shipping addresses, etc?

Agreed on the shipping addresses - I've updated the merge request to add the PersonShippingAddresses table.

Thanks for calling that out!

> Do you think we might want to break out a PersonStreetAddress table to handle things like separate billing/shipping addresses, multiple shipping addresses, etc? Agreed on the shipping addresses - I've updated the merge request to add the `PersonShippingAddresses` table. Thanks for calling that out!
sundog added 1 commit 2024-01-03 13:13:27 -04:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b setup-initial-schema main
git pull origin setup-initial-schema

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff setup-initial-schema
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MountainTownTechnology/aural_isle#10
No description provided.
Delete Branch "setup-initial-schema"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?