Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
    Luis Miguel Mejía Suárez
    @BalmungSan

    @tjarvstrand FYI neotypes/neotypes#359

    /cc @dimafeng

    Dmitry
    @dimafeng
    @BalmungSan it's taking a bit longer that I thought. Should we temporarily revert the release notes to not confuse people?
    Luis Miguel Mejía Suárez
    @BalmungSan
    Are they already up?
    Dmitry
    @dimafeng
    Doesn't it update the site on merge? I actually don't see the release notes for 0.18.0
    Luis Miguel Mejía Suárez
    @BalmungSan
    Remember we made that it only uploads the site of the last tag.
    Dmitry
    @dimafeng
    Oh, right!
    Luis Miguel Mejía Suárez
    @BalmungSan
    :+1:
    Dmitry
    @dimafeng
    @BalmungSan I think I broke 0.18.0 release :( I did a test release and it went through but I decided to re-run the whole publishing process removing the previously published artifacts. Now it doesn't allow me to publish 0.18.0 anymore. Would you be ok if I release it as 0.18.1 or 0.19.0?
    Luis Miguel Mejía Suárez
    @BalmungSan
    I think 0.18.1 would be ok :)
    We just need to fix the release notes.
    And add a note to 0.18.0 saying this version was published by accident.
    I saw something similar with http4s the other day, no worries :)
    Dmitry
    @dimafeng
    Great! Let me try to not screw up 0.18.0 :grinning:
    Luis Miguel Mejía Suárez
    @BalmungSan
    :grimacing:
    Dmitry
    @dimafeng
    0.18.1 has been released!
    Luis Miguel Mejía Suárez
    @BalmungSan
    :tada:
    Luis Miguel Mejía Suárez
    @BalmungSan
    Hi @dimafeng for some reason the site is not picking the 0.18.1_release_notes tag but rather some of the previous ones :open_mouth:
    Mayeb we would need to push an additional commit for it to re-deploy?
    Thomas Järvstrand
    @tjarvstrand
    Hi! How does derivation of QueryArgMappers for tuples work? I'm getting Could not find the QueryArgMapper for Seq[(String, String)] and I can't figure out how it's supposed to work
    Luis Miguel Mejía Suárez
    @BalmungSan
    How would that look like in a query? Like you want the first element of the tuple the be the field name?
    If so that doesn't work yet AFAIK.
    Thomas Järvstrand
    @tjarvstrand
    The way I thought it would work is that it would become a list of lists
    Luis Miguel Mejía Suárez
    @BalmungSan
    AFAIK neo4j doesn't support that.
    Thomas Järvstrand
    @tjarvstrand
    Ok. What does the line "Tuple(1-22)" mean in the list of supported types?
    Maybe it has too many check marks?
    Luis Miguel Mejía Suárez
    @BalmungSan

    Maybe it has too many check marks?

    Probably :sweat_smile:

    Sorry
    Thomas Järvstrand
    @tjarvstrand
    No worries. Would it work with HList?
    Luis Miguel Mejía Suárez
    @BalmungSan
    Actually no idea, but again how to do you want the query to look like?
    Nested lists are not supported by neo4j AFAIK.
    Thomas Järvstrand
    @tjarvstrand

    I have a case class:

    case class Tag(id: Id, name: String, wikiDataId: Option[Id], wikipediaUrl: Option[AbsoluteUrl])

    And I'm trying to accomplish something like this:

    FOREACH (tag IN $tags | 
              MERGE (t: Tag {id: tag[0]})
              ON CREATE
                SET 
                  t.name = tag[1],
                  t.wikiDataId = tag[2],
                  t.wikipediaUrl = tag[3],
                  t.createdAt = $createdAt
            )
    Using a nested list seems to work
    Luis Miguel Mejía Suárez
    @BalmungSan
    Ah yeah neo4j does support nested lists but not to store them.
    Can't you access named properties of the tag object?
    Like doesn't tag.id work?
    Thomas Järvstrand
    @tjarvstrand
    No. neotypes is unable to locate a QueryArgMapper for Tag even if I define one explicitly
    (I have mappers defined for Id and AbsoluteUrl)
    Ah, I just realized I was trying to derive the QueryArgMapper generically and it makes sense that that wouldn't work
    Luis Miguel Mejía Suárez
    @BalmungSan
    So was it fixed or not?
    Thomas Järvstrand
    @tjarvstrand
    Yeah, converting the Seq[Tag] to Seq[Seq[String]] works
    I don't think there's a generic way for neotypes to convert a case class into something supported as a query parameter in cypher
    Luis Miguel Mejía Suárez
    @BalmungSan

    Hi @dimafeng all ready for a new 0.18.2 release! neotypes/neotypes#363

    cc @tjarvstrand

    Dmitry
    @dimafeng
    Releasing!
    Luis Miguel Mejía Suárez
    @BalmungSan
    :tada:
    Luis Miguel Mejía Suárez
    @BalmungSan
    Hi @dimafeng thanks for the quick release!
    However, the site is still using the 0.18.0 version, why?
    Dmitry
    @dimafeng
    Hm, apperently something got broke :( I'll take a look soon
    Luis Miguel Mejía Suárez
    @BalmungSan
    Ty :+1: