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.

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

[feature/order] 주문 정보 가져오는 부분 수정 및 enum에서 Integer값을 가져오고 저장하도록 변경 #218

Merged
merged 6 commits into from
Nov 15, 2022

Conversation

chd830
Copy link
Member

@chd830 chd830 commented Nov 15, 2022

개요

enum을 저장할 때 index0부터 들어가는데 음수 값을 넣어야 하기 때문에 Integer로 수정하는 작업 수행

작업한 내용

  1. Change Enum to Integer
  2. 사용자의 id를 통해서 주문정보를 가져오고 그 주문에 일치하는 상세주문을 가져오는 부분으로 수정

테스트 결과

image

이슈번호

[#176]

@chd830 chd830 added the Feat 새로운 기능 추가 label Nov 15, 2022
@chd830 chd830 self-assigned this Nov 15, 2022
@chd830 chd830 changed the title [feature/order] 주문 정보 가져오는 부분 수정 및 enum에서 Integer값을 가져오고 저장하도록 변경 [WIP] [feature/order] 주문 정보 가져오는 부분 수정 및 enum에서 Integer값을 가져오고 저장하도록 변경 Nov 15, 2022
Comment on lines 1 to 20
//package com.feelmycode.parabole.global.config;
//
//import org.springframework.context.annotation.Configuration;
//import org.springframework.web.servlet.config.annotation.CorsRegistry;
//import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
//
//@Configuration
//public class WebMvcConfig implements WebMvcConfigurer {
// private final long MAX_AGE_SECS = 3600;
//
// @Override
// public void addCorsMappings(CorsRegistry registry) {
// registry.addMapping("/api/v1/**")
// .allowedOrigins("*")
// .allowedMethods("*")
// .allowedHeaders("*")
// .exposedHeaders("*")
// .maxAge(MAX_AGE_SECS);
// }
//}
package com.feelmycode.parabole.global.config;

import org.springframework.context.annotation.Configuration;
import org.springframework.web.servlet.config.annotation.CorsRegistry;
import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;

@Configuration
public class WebMvcConfig implements WebMvcConfigurer {
private final long MAX_AGE_SECS = 3600;

@Override
public void addCorsMappings(CorsRegistry registry) {
registry.addMapping("/api/v1/**")
.allowedOrigins("*")
.allowedMethods("*")
.allowedHeaders("*")
.exposedHeaders("*")
.maxAge(MAX_AGE_SECS);
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 수정하시면 안돼요오옹

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 수정햇서용

@chd830 chd830 changed the title [WIP] [feature/order] 주문 정보 가져오는 부분 수정 및 enum에서 Integer값을 가져오고 저장하도록 변경 [feature/order] 주문 정보 가져오는 부분 수정 및 enum에서 Integer값을 가져오고 저장하도록 변경 Nov 15, 2022
Copy link
Member

@jungeu1509 jungeu1509 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다~

Copy link
Member

@IMHYEWON IMHYEWON left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍👍👍👍

@chd830 chd830 merged commit 94d679d into dev Nov 15, 2022
@chd830 chd830 deleted the feature/order branch November 15, 2022 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 새로운 기능 추가
Projects
Development

Successfully merging this pull request may close these issues.

3 participants