Skip to content
This repository has been archived by the owner on Nov 10, 2020. It is now read-only.

Explore > Jobs #945

Merged
merged 4 commits into from
Dec 2, 2015
Merged

Explore > Jobs #945

merged 4 commits into from
Dec 2, 2015

Conversation

shawnbot
Copy link
Contributor

@shawnbot shawnbot commented Dec 1, 2015

.data(features, getter('id'));

items.exit().remove();
var enter = items.enter()

Choose a reason for hiding this comment

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

'enter' is defined but never used.

@shawnbot shawnbot changed the title [WIP] Explore > Jobs Explore > Jobs Dec 2, 2015
@meiqimichelle
Copy link
Contributor

Let's merge this and pull out the remaining issue (total for country showing up on state filter) as a separate issue. 🐭 💥

meiqimichelle added a commit that referenced this pull request Dec 2, 2015
@meiqimichelle meiqimichelle merged commit 3838b00 into dev Dec 2, 2015
@shawnbot
Copy link
Contributor Author

shawnbot commented Dec 2, 2015

Thanks, @meiqimichelle! Filed #953 and #952.

@shawnbot shawnbot deleted the explore-jobs branch December 10, 2015 17:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants