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.

Sign up for GitHub

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

Jump to bottom

Testbranch #1

Merged
merged 10 commits into from
Aug 20, 2021
Merged

Testbranch #1

merged 10 commits into from
Aug 20, 2021

Conversation

gbybk945
Copy link
Owner

@gbybk945 gbybk945 commented Aug 11, 2021 •
edited
Loading

概要

解説コメントを追加。

完了条件

  • 章ごとにコミット単位でまとまっている
  • チュートリアル xページの内容まで追記

technicakidz
technicakidz reviewed Aug 12, 2021
View reviewed changes
Copy link

@technicakidz technicakidz left a comment •
edited
Loading

Choose a reason for hiding this comment

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

PRありがとうございます，以下に指摘事項をあげます，

  • PRのタイトルを意味のわかるタイトルに変更しましょう（PRが途中の場合 ex. [wip] コード解説の追加 ）
  • コメントアウトの際は # とコメントの間に半角スペースを挟むとマークダウンにコピペする際に良いです
  • 以下が記載されている概要欄に完了条件を追記しましょう

解説コメントを追加。
ex.

### 概要
解説コメントを追加。

### 完了条件
- [ ] 章ごとにコミット単位でまとまっている
- [ ] チュートリアル xページの内容まで追記

requirements.txt Outdated

#ブログアプリに必要なライブラリをインストールするためのテキストコード
#pip install -r requirements.txt　をターミナルで起動しインストール
Copy link

Choose a reason for hiding this comment

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

インストール手順ですが，アプリの起動/停止等操作手順を README.md に追記し，まとめると良いです．

Copy link
Owner Author

Choose a reason for hiding this comment

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

追記してみました。

@gbybk945
Copy link
Owner Author

gbybk945 commented Aug 12, 2021 •
edited
Loading

以下が記載されている概要欄に完了条件を追記しましょう

解説コメントを追加。
ex.

概要

解説コメントを追加。

完了条件

  • 章ごとにコミット単位でまとまっている
  • チュートリアル xページの内容まで追記

↑こちらの内容についてですが、「概要欄」とはどこの部分を示しますか？

@gbybk945
Copy link
Owner Author

gbybk945 commented Aug 12, 2021 •
edited
Loading

再度、Pull Requestにあげてみました。

また、実際にpython3 manage.pyで以下のように実行し、「http://127.0.0.1:5000/」にアクセスしてみたのですが、HTMLのコードしか表示されず、CSSがうまく反映されていないような気がしました。なにか原因が考えられますか？

hosokawadaisuke@hosokawadaisukenoMacBook-Pro tutorial % python3 manage.py
/Users/hosokawadaisuke/Library/Python/3.8/lib/python/site-packages/flask_sqlalchemy/__init__.py:872: FSADeprecationWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future.  Set it to True or False to suppress this warning.
  warnings.warn(FSADeprecationWarning(
 * Serving Flask app 'flaskr' (lazy loading)
 * Environment: production
   WARNING: This is a development server. Do not use it in a production deployment.
   Use a production WSGI server instead.
 * Debug mode: on
 * Running on http://127.0.0.1:5000/ (Press CTRL+C to quit)
 * Restarting with stat
/Users/hosokawadaisuke/Library/Python/3.8/lib/python/site-packages/flask_sqlalchemy/__init__.py:872: FSADeprecationWarning: SQLALCHEMY_TRACK_MODIFICATIONS adds significant overhead and will be disabled by default in the future.  Set it to True or False to suppress this warning.
  warnings.warn(FSADeprecationWarning(
 * Debugger is active!
 * Debugger PIN: 796-463-616
127.0.0.1 - - [13/Aug/2021 06:16:51] "GET / HTTP/1.1" 200 -
127.0.0.1 - - [13/Aug/2021 06:16:51] "GET /favicon.ico HTTP/1.1" 404 -
127.0.0.1 - - [13/Aug/2021 06:17:32] "GET / HTTP/1.1" 200 -

↓表示結果

<!doctype html>

<title>Flaskr</title>

<link rel=stylesheet type=text/css href="/static/style.css">

<div class=page>

  <h1>Flaskr</h1>

 

 

  <form action="/add" method=post class=add-entry>

    <dl>

      <dt>Title:

      <dd><input type=text size=20 name=title>

      <dt>Text:

      <dd><textarea name=text rows=5 cols=20></textarea>

      <dd><input type=submit value=Share>

    </dl>

  </form>



  <ul class=entries>

 

    <li><em>Unbelievable.  No entries here so far</em>

 

  </ul>


</div>

@technicakidz
Copy link

READMEの一例

tutorial

このアプリについて

Flask チュートリアルの練習用リポジトリ

https://study-flask.readthedocs.io/ja/latest/02.html

Usage(使い方)

$ pip install -r requirements.txt

をターミナルで起動しインストール

requirements.txtについて

  • ブログアプリに必要なライブラリをインストールするためのテキストコード

その他

  • oo は完全に完成していません

@gbybk945 gbybk945 merged commit 2809606 into main Aug 20, 2021
@technicakidz technicakidz mentioned this pull request Aug 25, 2021
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@technicakidz technicakidz technicakidz left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gbybk945 @technicakidz