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

remove empty values from transactions originating in form builder #21

Open
trevor opened this issue Jun 28, 2013 · 1 comment
Open

remove empty values from transactions originating in form builder #21

trevor opened this issue Jun 28, 2013 · 1 comment

Comments

@trevor
Copy link
Contributor

trevor commented Jun 28, 2013

When using the "+" symbol to add fields, stray fields may be left and shouldn't be a part of the transaction.

an example with a stray field:

[ 
 {":db/id" #db/id [":db.part/db"]
  ":db/ident" ":myname/enum1"}

 {":db/id" #db/id [":db.part/db"]
  ":db/ident" ":myname/enum2"}

 {":db/id" #db/id [":db.part/db"]
  ":db/ident" ":myname/null"}
]
@shaunxcode
Copy link
Owner

Good find, this is actually being fixed in the feature release this
weekend! You will also notice it is making keywords strings.. yuck. It is
now using the newer version of jsedn which cleans that up via first-class
keyword usage etc. I will keep this open and close it when I push out the
new version!

On Fri, Jun 28, 2013 at 7:52 AM, Trevor Wennblom
notifications@github.comwrote:

When using the "+" symbol to add fields, stray fields may be left and
shouldn't be a part of the transaction.

an example with a stray field:

[
{":db/id" #db/id [":db.part/db"]
":db/ident" ":myname/enum1"}

{":db/id" #db/id [":db.part/db"]
":db/ident" ":myname/enum2"}

{":db/id" #db/id [":db.part/db"]
":db/ident" ":myname/null"}]


Reply to this email directly or view it on GitHubhttps://github.com//issues/21
.

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

No branches or pull requests

2 participants