Ramnivas Laddad ramnivas

ramnivas commented on issue tpolecat/doobie#200
@ramnivas

@tpolecat This is just to remind that you mentioned during Scala By the Bay that there may be an easier way to deal with such situations.

@ramnivas

I remember thinking about this problem and while I didn't implement it at that time, I was thinking along the lines of testing for an environment v…

ramnivas commented on issue tpolecat/doobie#200
@ramnivas

Yes. I am going with the .map approach for now. Separately, I am considering making ids simple Long (or other primitive types such as String as ap…

ramnivas commented on issue tpolecat/doobie#200
@ramnivas

Looks like the problem you mentioned can come up easily. I tried to use this workaround in a real project which have ids defined as Option[Long]. S…

@ramnivas
compilation error
@ramnivas

Fixed via 52d4d08 Looks like scala-reflect is need to deal with macros in the example project.

@ramnivas
  • @ramnivas 52d4d08
    Revert "Remove unneeded scala-reflect dependency"
ramnivas commented on issue tpolecat/doobie#189
@ramnivas

This works! Thanks for your quick response. It will be nice to have this info somewhere in the book of doobie (I saw the last section of http://tp…

@ramnivas
Allow splicing literal values in sql interpolator
@ramnivas
  • @ramnivas cb5da4a
    Remove unneeded scala-reflect dependency
@ramnivas
  • @ramnivas 9cdf7bf
    Remove unneeded scala-reflect dependency
@ramnivas

Here is an outline of how we might do it. Given a component with the following as one of its case class parameter: onChange: js.UndefOr[Seq[String]…

@ramnivas
  • @ramnivas c732b64
    Make it compatible scalajs-react-bridge snapshot
ramnivas commented on issue scala-js/scala-js#1743
@ramnivas

@sjrd Done. It is indeed an improvement over the current code.

@ramnivas
  • @ramnivas 8bc3b80
    Use js.Dictionary to simplify Map conversion.
@ramnivas

@chandu0101 Your use case is now fixed in a more general manner (see 3be28c9). See tests to see how it can be used.

@ramnivas
Check for the key type before translating Map property
@ramnivas

Fixed through 3be28c9

@ramnivas
Move more code to compile-time
@ramnivas

Fixed through 3be28c9

@ramnivas
  • @ramnivas 3be28c9
    Allow conversion of custom types.
@ramnivas

I believe I will have to raise the priority for #2. By the time we reach runtime, erasure has removed too much info for us to do anything too inter…

ramnivas commented on issue scala-js/scala-js#1743
@ramnivas

Just a bit more info. If I change the line from js.Dynamic.literal.applyDynamic("apply")(map.toSeq: _*) to js.Dynamic.literal(map.toSeq: _*) I get

@ramnivas
scala classes as parameter types
@ramnivas

Fixed via 67f80b0. Thanks @chandu0101

@ramnivas
accept scala classes as params types
ramnivas commented on pull request payalabs/scalajs-react-bridge#7
@ramnivas

Merged via 67f80b0