Fixed
Status Update
Comments
yb...@google.com <yb...@google.com>
su...@gmail.com <su...@gmail.com> #2
Did anyone get an answer to this question?
el...@google.com <el...@google.com>
ap...@google.com <ap...@google.com> #3
Project: platform/frameworks/support
Branch: androidx-master-dev
commit 5b6cb684c06bc5be3aa3dc072d9c73fab77a537a
Author: Elif Bilgin <elifbilgin@google.com>
Date: Wed Sep 16 14:41:24 2020
Update to TypeConverters documentation to clarify that TypeConverters can only be used to convert columns / fields and not rows.
Test: n/a
Bug: 77307836
Relnote: Update to TypeConverters documentation to clarify that TypeConverters can only be used to convert columns / fields and not rows.
Change-Id: I07c56c02866b27e8e2e4802f2426130f8713f472
M room/common/src/main/java/androidx/room/TypeConverters.java
https://android-review.googlesource.com/1428673
Branch: androidx-master-dev
commit 5b6cb684c06bc5be3aa3dc072d9c73fab77a537a
Author: Elif Bilgin <elifbilgin@google.com>
Date: Wed Sep 16 14:41:24 2020
Update to TypeConverters documentation to clarify that TypeConverters can only be used to convert columns / fields and not rows.
Test: n/a
Bug: 77307836
Relnote: Update to TypeConverters documentation to clarify that TypeConverters can only be used to convert columns / fields and not rows.
Change-Id: I07c56c02866b27e8e2e4802f2426130f8713f472
M room/common/src/main/java/androidx/room/TypeConverters.java
Description
Version used: 1.1.0
Devices/Android versions reproduced on: n/a
Ideally, the JavaDocs for TypeConverters would be a bit more specific about what things can be converted via TypeConverters. For example, there is an indication that you cannot use TypeConverters to convert the return value of a @Query-annotated DAO method:
One possibility would be to convert the bulleted list in to a table, with rows for the @TypeConverters location (e.g., "put it on a Database"), columns for what specific things get affected (entity fields, DAO method parameters, etc.), and X's or other indicators in the cells to show what is and is not covered.
Thanks for considering this!