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

add update and details features #49

Merged
merged 2 commits into from
Mar 11, 2016
Merged

Conversation

lluminita
Copy link
Owner

solves issues: #45 #44 #43

Former closed PR: #46

@lluminita lluminita force-pushed the add-update-and-detail-views branch from ea40083 to 5fd0e88 Compare March 9, 2016 08:56
<h2 class="section-heading">{{ project.title }}</h2>
<p>{{ project.created_at|date:"D d M Y" }}</p>
<div
{% if project.difficulty == "beginner" %}
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can do
<div class=glossy-{{ project.difficulty }}>

@lluminita
Copy link
Owner Author

I fixed the difficulty comment and I remove the create and update templates and replaced with project_form, this way there is no need to use the attribute template_name in the views.

@lluminita lluminita force-pushed the add-update-and-detail-views branch from 591c8c2 to 22955f0 Compare March 9, 2016 09:48
@lluminita lluminita force-pushed the add-update-and-detail-views branch from 22955f0 to f107e1b Compare March 9, 2016 10:23
@mei-li
Copy link
Collaborator

mei-li commented Mar 9, 2016

I made this issue for the owner #52

@mei-li
Copy link
Collaborator

mei-li commented Mar 9, 2016

👍

lluminita added a commit that referenced this pull request Mar 11, 2016
@lluminita lluminita merged commit a8e1dc2 into master Mar 11, 2016
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