-
Notifications
You must be signed in to change notification settings - Fork 122
Conversation
<i18n-msg msgid="attend-onsite">Attend onsite</i18n-msg> | ||
</a> | ||
</paper-tab> | ||
<paper-tab> | ||
<paper-tab label="offsite"> | ||
<a href="offsite.html" layout horizontal center> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data-ajax-link
<i18n-msg msgid="attend-onsite">Attend onsite</i18n-msg> | ||
</a> | ||
</paper-tab> | ||
<paper-tab> | ||
<paper-tab label="offsite"> | ||
<a href="offsite.html" layout horizontal center> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
data-ajax-link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's no partial for registration yet, so i'd rather get a simple 404 here.
Reviewed |
PTAL |
LGTM. I'm going to wait for the CI to see if it's working. Otherwise, will merge |
I checked CI with Alex. there are bower-related problems, but probably nothing related to this PR in particular. |
👍 from a service worker caching perspective. Nifty that you all got this working! |
Actually, I think we want: |
Going to merge this. I'll follow up with a title cl. |
Adding router for ajaxified links
This is based on Eric's #92, with some additional changes added on top of it.
What got changed from original PR: