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

Block for extra content in area/postcode info #284

Merged
merged 1 commit into from
Feb 17, 2017
Merged

Conversation

zarino
Copy link
Member

@zarino zarino commented Feb 15, 2017

Useful for cobrands that want to display a small amount of custom data, or a message to users, without overriding the whole template.

(We’re using this in MapIt UK to display a message about business users and paid plans.)

Useful for cobrands that want to display a small amount of custom
data, or a message to users, without overriding the whole template.
Copy link
Member

@dracos dracos left a comment

Choose a reason for hiding this comment

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

One possible naming change is all 🍇

{% url json_view postcode=postcode.postcode as json_url %}
<p>{% blocktrans %}Get <a href="{{ json_url }}">this data as JSON</a>{% endblocktrans %}</p>

{% block area_info_extra %}{% endblock %}
Copy link
Member

Choose a reason for hiding this comment

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

Should this be postcode_info_extra?

Copy link
Member Author

Choose a reason for hiding this comment

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

I considered it, but then the HTML element it appears in is called area_info (even on the postcode page) – so I stuck with area_info_extra because it's extra content to go in the .area_info element.

Copy link
Member

Choose a reason for hiding this comment

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

Yeah, I noticed that after (and merged+deployed as you see) :)

@dracos dracos merged commit 3d26447 into master Feb 17, 2017
@dracos dracos removed the Reviewing label Feb 17, 2017
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