merch backend #10
No reviewers
Labels
No labels
Confirmed
Confirmed
Confirmed
Duplicate
Confirmed
Invalid
Confirmed
Review Needed
Confirmed
Won't Fix
Digital Dash
JavaScript
React
JavaScript
Svelte
Kind
Bug
Kind
Enhancement
Kind
Feature
Kind
Security
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Role
All Members
Role
Future Software Lead
Role
Intermediate Member
Role
Junior Member
Role
Senior Member
Role
Software Lead
Size
L
Size
M
Size
S
Size
XL
Size
XS
Status
Abandoned
Status
Blocked
Status
Help wanted
Status
Need More Info
Status
Question
Tech
Actions
Tech
C
Tech
C++
Tech
CSS
Tech
Docker
Tech
Draw.io
Tech
Golang
Tech
HTML
Tech
JavaScript
Tech
Python
Tech
Rust
Tech
Terraform
UI
Component
UI
Layout
UI
Page
Work Type
Design Work
Work Type
Development
Work Type
Documentation
Work Type
Review
Work Type
Testing
No milestone
No project
No assignees
2 participants
Notifications
Due date
Depends on
#1 Merch Shop Backend
ucalgary-baja/ucalg-baja-cloud
Reference
ucalgary-baja/ucalg-baja-cloud!10
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "merch"
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?
Do OpenAPI/swagger docs
merchto WIP: merchWIP: merchto WIP: merch backend@ -6,3 +10,4 @@use utoipa_swagger_ui::SwaggerUi;#[actix_web::main]async fn main() -> std::io::Result<()> {Differentiate if not in docker container
Add the notes to the appropriate files
@ -1,13 +1,20 @@use actix_cors::Cors;use actix_web::{App, HttpServer, web};use tokio::sync::Mutex;Check if actixs has their own mutex
no built in version into actixs web
we are using tokio for the .await on the mutex
@ -20,3 +27,3 @@//.supports_credentials()).service(hello).app_data(database.clone())why are you cloning here that is a mistake
that may be needed for ownership rules, if it is a Arc then a hard ref is needed here
actix_web::web::Data is a wrapper around Arc, therefore the clone is a O(1) an increase the amount of references it has (its minimal performance loss)
@ -2,2 +6,4 @@use darkicewolf50_actix_setup::log_incoming;use serde::{Deserialize, Serialize};use serde_json::json;use tokio::sync::Mutex;again check if actixs has a mutex type
see previous comment about this
@ -30,0 +181,4 @@.set_value(customer_info.shipping_details.clone());writer::xlsx::write(&book, self.connection.as_ref().unwrap().as_path()).unwrap();}change to spread out shipping details in excel sheet
add comment like above for the section of what it is doing
Shipping detail rework to spread out
commented here write out
missing log output
its there just not in the preview
Remove pull request from build & upload from pull request
Remove duplicate tests for all tests and doc tests
a3dc02dee6e356e90a02Update actions to build with rust-alpine version instead of Ubuntu
Make UUID optional, then give a UUID if None to each order
OpenAPI docs will be completed with ucalgary-baja/ucalg-baja-cloud#8 as we don't have time now
Change to include comment fixes
@ -14,1 +13,3 @@sub_team: String,#[derive(Debug, Clone, Serialize, Deserialize)]pub struct CustomerInfo {order_id: Option<ArcString>,When enum is none it should be given a uuid
@ -30,0 +33,4 @@}#[get("/recieve_order/{order_request}")]pub async fn recieve_order(order_request: web::Json<OrderRequest>) -> impl Responder {make items in this pub so that the order uuid can be edited to include a uuid when enum is None
WIP: merch backendto merch backendGG
Merged into dev
Merged into dev