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

Allow tags for Accounts, Addresses and Contracts #2712

Merged
merged 14 commits into from
Oct 19, 2016
Merged

Allow tags for Accounts, Addresses and Contracts #2712

merged 14 commits into from
Oct 19, 2016

Conversation

ngotchac
Copy link
Contributor

[Copy of PR #2697]

Closes #2643

This PR adds tagging capabilities to Accounts, Addresses and Contracts. Multiple can be added per item, and these can be searched. They can also be sorted by name or by tag.

Currently, the search accepts multiple tokens, that are joined with OR ; eg. if item 1 has tag foo and item 2 has tag bar, as search for the two tokens foo and bar will display item 1 and 2.
This can be changed easily to an AND behavior.

See it in action here: https://youtu.be/3UA-QI0G_UQ

@ngotchac ngotchac added the A0-pleasereview 🤓 Pull request needs code review. label Oct 19, 2016
@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Ethcore CLA Bot

@@ -131,6 +132,16 @@ module.exports = {
]
}),
new HappyPack({
id: 'less',
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we still need this now?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So it's not used by any file, but it was there previously, so not sure...
Should we get completely rid of LESS ? (ie. in Webpack and package.json)

Copy link
Contributor

Choose a reason for hiding this comment

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

If we are not using it, yes, best approach.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, will do

@jacogr jacogr added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 19, 2016
@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 05cdecc on ng-ui-tags into * on master*.

@gavofyork gavofyork merged commit cc10f41 into master Oct 19, 2016
@gavofyork gavofyork deleted the ng-ui-tags branch October 19, 2016 09:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants