refactor link_column

Author: sinisaos

README.md

@@ -1,4 +1,4 @@
-Experimenting with using a dockerized Piccolo Admin with an existing (legacy) database.
+Dockerized Piccolo Admin to use existing (legacy) database.
 
 ### Usage
 
@@ -41,12 +41,12 @@ tables:
   # An example of additional Piccolo Admin configuration
   Actor:
     visible_columns:
-      - actor_id
       - first_name
     visible_filters:
       - actor_id
       - first_name
     menu_group: Movies
+    link_column: first_name
   Address:
     visible_columns:
       - address_id
@@ -57,6 +57,7 @@ tables:
       - address
       - city_id
     menu_group: Location
+    rich_text_columns: address
   City:
     visible_columns:
       - city_id
@@ -73,6 +74,10 @@ tables:
       - country_id
       - country
     menu_group: Location
+
+sidebar_links:
+  Piccolo Admin: https://piccolo-admin.readthedocs.io/en/latest/index.html
+  Piccolo ORM: https://piccolo-orm.readthedocs.io/en/latest/index.html
 ```
 
 For these changes to take effect, you must stop the container and rebuild it with.

app/main.py

@@ -119,12 +119,17 @@ async def main():
             except TypeError:
                 rich_text_columns = None
             # link column
-            link_column = [
-                column
-                for column in table._meta.columns
-                if column._meta.name
-                == BASE_CONFIG[capitalize_table_name].get("link_column", None)
-            ]
+            try:
+                link_column = [
+                    column
+                    for column in table._meta.columns
+                    if column._meta.name
+                    == BASE_CONFIG[capitalize_table_name].get(
+                        "link_column", None
+                    )
+                ][0]
+            except IndexError:
+                link_column = None
             # menu_group
             menu_group = BASE_CONFIG[capitalize_table_name].get(
                 "menu_group", None
@@ -136,7 +141,7 @@ async def main():
                     visible_columns=visible_columns,
                     visible_filters=visible_filters,
                     rich_text_columns=rich_text_columns,
-                    link_column=None if link_column == [] else link_column[0],
+                    link_column=link_column,
                     menu_group=menu_group,
                 )
             )

config.yaml

@@ -1,41 +1,4 @@
 tables:
-  # An example of additional Piccolo Admin configuration
-  # Actor:
-  #   visible_columns:
-  #     - actor_id
-  #     - first_name
-  #   visible_filters:
-  #     - actor_id
-  #     - first_name
-  #   menu_group: Movies
-  #   link_column: first_name
-  # Address:
-  #   visible_columns:
-  #     - address_id
-  #     - address
-  #     - city_id
-  #   visible_filters:
-  #     - address_id
-  #     - address
-  #     - city_id
-  #   menu_group: Location
-  #   rich_text_columns: address # it only works in the Text type column
-  # City:
-  #   visible_columns:
-  #     - city_id
-  #     - city
-  #   visible_filters:
-  #     - city_id
-  #     - city
-  #   menu_group: Location
-  # Country:
-  #   visible_columns:
-  #     - country_id
-  #     - country
-  #   visible_filters:
-  #     - country_id
-  #     - country
-
+  # Additional tables configurations go here
 sidebar_links:
-  # Piccolo Admin: https://piccolo-admin.readthedocs.io/en/latest/index.html
-  # Piccolo ORM: https://piccolo-orm.readthedocs.io/en/latest/index.html
+  # External links go here