Bummer: The for loop should be between </h1> and <div>.

Question

What does this error mean ? Isn't it already in between </h1> and <div> ?

app.py

from models import db, Plant, app
from flask import render_template, url_for, request, redirect


@app.route('/')
def index():
    plants = Plant.query.all()
    return render_template('index.html', plants=plants)


@app.route('/form', methods=['GET', 'POST'])
def form():
    if request.form:
        new_plant = Plant(plant_type=request.form['type'], 
                               plant_status=request.form['status'])
        db.add(new_plant)
        db.commit()
        return redirect(url_for('index'))
    return render_template('form.html')


if __name__ == '__main__':
    app.run(host='0.0.0.0', port=8000)

models.py

from flask import Flask
from flask_sqlalchemy import SQLAlchemy


app = Flask(__name__)
app.config['SQLALCHEMY_DATABASE_URI'] = 'sqlite:///plants.db'
db = SQLAlchemy(app)

class Plant(db.Model):
    id = db.Column(db.Integer, primary_key=True)
    plant_type = db.Column(db.String())
    plant_status = db.Column(db.String())

form.html

{% extends 'layout.html' %}

{% block content %}
    <form action="{{ url_form('form') }}" method="POST">
        <label for="type">Plant Type:</label>
        <input type="text" name="type" id="type" placeholder="Plant type">

        <p>Plant Status:</p>
        <label for="good">GOOD</label>
        <input type="radio" name="status" id="good">
        <label for="ok">OK</label>
        <input type="radio" name="status" id="ok">
        <label for="bad">BAD</label>
        <input type="radio" name="status" id="bad">
    </form>
{% endblock %}

index.html

{% extends 'layout.html' %}

{% block content %}
    <h1>Our Plants:</h1>

    {% for plants in plants %}
    <div>
        <h2>Plant Type</h2>
        <p>Plant Status</p>
    </div>
    {% endfor %}

{% endblock %}

Answer 1 · 2024-01-22T16:44:33Z

January 22, 2024 4:44pm

Hey Jorge Grajeda 👋

You'll want to have a look at how you're naming the variable you're iterating over. Currently you're giving it the same name as the list of plants instead of giving it a single plant name. This is what throws the tests off 🙂

After that adjustment you'll also want to have a look at the h2 and p text contents as they are currently the static strings of "Plant Type" and "Plant Status" while the challenge asks to replace those with the values for the specific plant you're iterating over 🙂

Hope this helps! 😃

Welcome to the Treehouse Community

Looking to learn something new?

Jorge Grajeda

Jorge Grajeda

Bummer: The for loop should be between </h1> and <div>.

1 Answer

Rohald van Merode

Rohald van Merode

Jorge Grajeda

Jorge Grajeda