Skip to content
This repository was archived by the owner on Oct 2, 2019. It is now read-only.

fix tagging for all and feat tagging-equals function #1562

Closed
wants to merge 4 commits into from

Conversation

Teegrounet
Copy link
Contributor

fix tagging for all (I see that this was fixed in PR #972, but I can't find that commit either in the 0.16.1 tag, nor master.)
add tagging-equals option to verify if element exist in list before insert without some properties of object
add some tests

@user378230
Copy link
Contributor

#972 has not been merged it still needs work I don't see a reason to include it in this PR though its not well related to the other change.

Would it make sense to just have a generic custom equals which can be used everywhere? Or is there a reason you only added it for tagging?

It looks like you also have issues with indents too.

@Teegrounet
Copy link
Contributor Author

Teegrounet commented Apr 16, 2016

I fix a problem with tagging label -> if it don't be initialized we can see : item + ' ' + undefined (single)
I fix a problem in watchCollection (you can see with refreshDelay) -> tagging element was removed when the list change (single and multi)

I try to optimize performance of tagging : i store the element because we spend time to search it in lists

I need this function to compare the element create by tagging (which can not contain all info) with items of the list.

For example :
I have a list of countries (object with an id of my DB and a name), when user search an element, the tagging element doesn't have id
-> we need to compare with country name and angular.equals can't be used.

@user378230
Copy link
Contributor

As previously stated there are issues with your indenting.

Also you need to rebase not merge.

Finally, if you're fixing bugs in the library can you submit this as a separate PR. This makes it easier to track and revert if necessary.

I suggest also you have a read of CONTRIBUTING.md

If you have any questions please ask. Thanks! 👍

@Teegrounet
Copy link
Contributor Author

I am sorry but I read the CONTRIBUTING.md but I don't understand all it write... It's very difficult to begin on GitHub...

I understand it's not a good thing to commit all in one (and without issues), but what do you want I do now ?

@user378230
Copy link
Contributor

Closing due to age/poor quality, sorry!

@user378230 user378230 closed this Aug 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants