velopert / learning-react

[길벗] 리액트를 다루는 기술 서적에서 사용되는 코드
567 stars 406 forks source link

[개정판] p560, 20장 3.5 정적 파일 제공하기 부분 manifest 객체 관련 오류 #255

Open awer000 opened 4 years ago

awer000 commented 4 years ago

안녕하세요. 벨로퍼트님. 좋은 책 잘 보고 있습니다.

다름이 아니라 p560에 build에 있는 js와 css의 정적 파일을 가져오는 곳에서 오류가 나서 말씀드립니다. 책에서는 아래와 같은 코드로 되어있는데, manifest의 객체를 가져오는 과정에서 자꾸 오류가 나서 보니까 뒤에 가져오고자 하는 값이 manifest가 아니라 manifest의 files 객체 내에 있더라구요. 그래서 manifest.files로 수정하니 잘 되었습니다.

오탈자에 이미 올라와 있는 내용인가 확인했는데 없길래, 이슈 남겨 둡니다.

const chunks = Object.keys(manifest.files)
  .filter(key => /chunk\.js$/.exec(key))
  .map(key => `<script src="${manifest[key]}"></script>`)
  .join("");

function createPage(root) {
  return `<!DOCTYPE html>
  <html lang="en">
  <head>
    <meta charset="utf-8 />
    <meta
      name="viewport"
      content="width=device-width,initial-scale=1,shrink-to-fit=no"
    />
    <meta name="theme-color" content="#000000 />
    <title>React App</title>
    <link rel="shortcut icon" href="/favicon.ico" />
    <link href="${manifest["main.css"]}" rel="stylesheet" />
  </head>
  <body>
    <noscript>You need</noscript>
    <div id="root">
      ${root}
    </div>
    <script src="${manifest["runtime-main.js"]}"></script>
    ${chunks}
    <script src="${manifest["main.js"]}"></script>
  </body>
  </html>
  `;
}
MellonHero commented 4 years ago

책 펴보고 확인해보니까 제 책에는 manifest.files로 잘 나와있네요.

초판 2쇄..? 여기서 수정이 된 부분 같습니다.

233

awer000 commented 4 years ago

책 펴보고 확인해보니까 제 책에는 manifest.files로 잘 나와있네요.

초판 2쇄..? 여기서 수정이 된 부분 같습니다.

확인 감사합니다. 제가 1쇄를 가지고 있어서 그 부분이 2쇄에서는 수정이 된 것 이었나 보네요 ㅎㅎ

velopert commented 4 years ago

https://github.com/velopert/learning-react/blob/master/_old_corrections/%EA%B0%9C%EC%A0%95%ED%8C%90-1%EC%87%84.md#2035-pg-560