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

9주차 Assignment - 류고은 #2

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

gosilver20012001
Copy link

Description

구현 및 작업에 대한 내용을 적어주세요.

  • CRUD와 media를 이용한 페이지 구현

Reference

참고한 레퍼런스가 있다면 적어주세요.

  • 'error: src refspec master does not match any'오류 검색

Question

어려웠던 점이 있거나 더 궁금한 점에 대해 적어주세요.

  • 오류 해결이 너무 어려워요

Copy link
Contributor

@staccato20 staccato20 left a comment

Choose a reason for hiding this comment

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

< Comment >

전체적으로 가이드라인을 꼼꼼하게 읽어주시지 않은 것 같아 아쉽습니다ㅠㅠ 그래도 과제 기한 조금 전에 제출해주셔서 정말 감사합니다8ㅅ8 우리 조금만 더 힘내봐요!

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

secrets.json이 ignore되지 않았습니다. 더불어 이미 올라간 secrets.json도 없애주세요!

goeun/db.sqlite3 Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

db가 ignore되지 않았습니다. 이전에 강의해드린 .gitignore 만드는 방식대로 만드셨다면 기본적으로 db가 ignore될 수 있게 작성되어 있어야 하지만 본 .gitignore은 그렇지 않습니다. 파일 생성 과정에서 실수가 없었는지 확인을 부탁드립니다! 더불어 이미 올라간 db도 없애주세요!

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

본 index.html은 다른 페이지들에 레이아웃을 상속하지 않고 있습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

index.html로부터 레이아웃을 상속받지 않고 있습니다.

Comment on lines +5 to +7
title = models.CharField(max_length=30)
pub_date = models.DateTimeField('data published')
weather = models.CharField(max_length=20)
Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

글자수가 제한되는지 직접 써보셨나요? 글자수가 제한되지 않고 있습니다! form을 사용하지 않았을 때는 글자수 제한이 model에 작성하는 것만으로는 되지 않는다는 사실을 지난 강의에서 배우셨을 것입니다!

힌트 : models.py이 아닌 다른 곳에서 글자수를 제한할 수 있는 방법을 찾아보시면 될 것 같습니다😉

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

index.html로부터 레이아웃을 상속받지 않고 있습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

index.html로부터 레이아웃을 상속받지 않고 있습니다.

Copy link
Contributor

@staccato20 staccato20 Jun 21, 2023

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

index.html로부터 레이아웃을 상속받지 않고 있습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

  1. U와 D 기능이 존재하지 않습니다.

  2. index.html로부터 레이아웃을 상속받지 않고 있습니다.

goeun/journalapp/models.py Outdated Show resolved Hide resolved
Comment on lines 36 to 38
{% block content2 %}

{% endblock %}
Copy link
Contributor

Choose a reason for hiding this comment

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

< Comment >

달리 활용된 곳이 없는 것 같은데 그냥 작성만 되어 있는 점이 아쉽습니다.

Comment on lines 8 to 11
<nav>
<a href="{% url 'index' %}">index</a>
<a href="{% url 'write' %}">일기 작성하기</a>
</nav>
Copy link
Contributor

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

url명이 잘못되어 있습니다.

Comment on lines 13 to 16
날씨: <input type="text" name="weather" maxlength="20" value="{{ediit_review.weather}}"> <br>
일기: <textarea name="content" cols="30" rows="10">{{ ediit_review.weather }}</textarea>
<div>
사진 업로드: <input type="file" name="phto" value="{{ediit_review.phto}}">
Copy link
Contributor

@staccato20 staccato20 Jun 22, 2023

Choose a reason for hiding this comment

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

< 가이드라인 미준수 >

수정하는 페이지에서 제목을 제외한 날씨, 일기 데이터를 가지고 오지 못하고 있습니다.

Copy link
Contributor

@staccato20 staccato20 left a comment

Choose a reason for hiding this comment

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

🥳 과제 통과입니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants