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

can-query-logic documentation #16

Open
indifferentghost opened this issue Sep 7, 2018 · 2 comments
Open

can-query-logic documentation #16

indifferentghost opened this issue Sep 7, 2018 · 2 comments

Comments

@indifferentghost
Copy link
Contributor

indifferentghost commented Sep 7, 2018

can-query-logic.html

can-query-logic.prototype.count.html

  • ES6
  • codepen
  • indentation
  • article is a stub
  • Link to MDN for [Infinity](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Infinity)?

can-query-logic.prototype.difference.html

  • ES6
  • codepen
  • indentation
  • article is a stub

can-query-logic.prototype.filterMembersAndGetCount.html

  • ES6
  • codepen
  • indentation
  • article is a stub

can-query-logic.prototype.filterMembers.html

  • codepen
  • indentation
  • article should have a more thorough explanation when super set is required.
  • Mention pagination and sorting and how it's used with filterMembers.

can-query-logic.prototype.identityKeys.html

  • codepen
  • ES6
  • indentation
  • I'm curious if there's any quirks or edge cases that identityKeys has.

can-query-logic.prototype.index.html

  • codepen
  • indentation
  • Added example if no sort option is given.
  • Add required to the params. Most other methods see if the given data structures are correct.
  • Since object key order is unreliable is it reliable to not provide a key to sort by?

can-query-logic.prototype.intersection.html

  • codepen
  • indentation

can-query-logic.prototype.isDefinedAndHasMembers.html

  • codepen
  • ES6
  • indentation
  • in examples the query object is returned if it is not a special object (that is not UNIVERSAL). Update documentation to reflect such.

can-query-logic.prototype.isEqual.html

  • codepen
  • indentation

can-query-logic.prototype.isMember.html

  • codepen
  • indentation

can-query-logic.prototype.isProperSubset.html

  • codepen
  • indentation
  • article is a stub, should have better example showing an example set and subset.

can-query-logic.prototype.isSpecial.html

  • codepen
  • indentation
  • ES6
  • example should use .isSpecial and not .isDefinedAndHasMembers.
  • changed example results and documentation as this does return true if the object is UNIVERSAL, EMPTY, UNDEFINABLE, or UNKNOWABLE.

can-query-logic.prototype.isSubset.html

  • codepen
  • indentation
  • should provide example where .isSubset is not true.

can-query-logic.prototype.memberIdentity

  • is hidden using @hide tag
  • ES6
  • codepen
  • indentation

can-query-logic.prototype.unionMembers.html

  • codepen
  • indentation

can-query-logic.prototype.union.html

  • codepen
    • codepen only logs up to a certain nesting added JSON.stringify around return value to show full object.
  • indentation

can-query-logic/comparison-operators.html

  • Fixed spelling mistake under $ne signature. $eq -> $ne.
  • Add example for each operator.

Discourse:

  • The folder [/operators](https://github.com/canjs/can-query-logic/tree/master/doc/operators) can probably be removed unless the eventual idea is to separate can-query-logic/comparison-operators.html.
  • Do the signatures need examples?

can-query-logic/query.html

  • codepen
  • indentation

can-query-logic.EMPTY.html

  • needs example
  • Article is a stub.

can-query-logic.KeysAnd.html

  • Article under doc/static/AndKeys.md it should probably be renamed to KeysAnd.md to match other references.
  • ES6
  • Codepenable
  • linked references to static-methods/set[...]

can-query-logic.UNDEFINABLE.html

  • ES6
  • Codepenable
  • commented out fragment

Use UNKNOWABLE to represent

Discourse:

  • An example on the article references QueryLogic.Not which isn't an operator. There's currently no @codepen tag on the example to prevent confusion. It might be worth while to remove this example.

can-query-logic.UNIVERSAL.html

  • Article needs an example.
  • Article is a stub.

can-query-logic.UNKNOWABLE.html

  • codepen
    • get alternatives example working
  • indentation

can-query-logic.set.difference.html

  • codepen
  • ES6
  • indentation

discourse:

  • QueryLogic.set.difference seems to be QueryLogic.difference should this be changed?

can-query-logic.set.intersection.html

  • codepen
  • es6
  • indentation

discourse:

  • QueryLogic.set.intersection seems to be QueryLogic.intersection should this be changed in all documentation?

can-query-logic.set.union.html

  • codepen
  • es6
  • indentation

discourse:

  • QueryLogic.set.union seems to be QueryLogic.union should this be changed in all documentation? This could be confused with QueryLogic.prototype.union is there a good way to separate these?

doc/value-types

Discourse:

  • value types is a folder with empty files, is this meant to continue to expand on various QueryLogic value types ( ex: QueryLogic.GreaterThan ) or should this folder be removed?

doc/etc

  • ES6
  • codepen
  • doesn't match formatting of other documents

discourse:

  • This file doesn't appear on main the main document page and should either be removed or updated to fit similar formatting.

can-query-logic.defineComparison.html

  • codepen
    • GreaterThan, LessThan, etc. are not global constants from the can framework, they do seem to be functions on the QueryLogic.
    • Updated Use to reference examples/recipe-example.js
  • indentation
@indifferentghost indifferentghost self-assigned this Sep 7, 2018
@indifferentghost indifferentghost changed the title can-query-logic (WIP) can-query-logic documentation Sep 21, 2018
@justinbmeyer
Copy link
Contributor

I think you might have forgotten about can-query-logic.defineComparison.html

@justinbmeyer
Copy link
Contributor

etc should be removed. Looks like you removed it: https://github.com/canjs/can-query-logic/pull/18/files#diff-0a90908bb7e02eebae809eb72a9c5207

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants