diff --git a/ayanova/devdocs/rando-todo.txt b/ayanova/devdocs/rando-todo.txt index 9b48df4e..0cc440a7 100644 --- a/ayanova/devdocs/rando-todo.txt +++ b/ayanova/devdocs/rando-todo.txt @@ -1,5 +1,55 @@ Grab bag stuff that didn't fit into official TODO + + +todo: missing feature, can't filter widgetlist by User due to no UserList being available + Is this meant to be text only filter and we have no id filter system? + How hard to add proper list selection and ID because it seems important? +thoughts: + I think it might be possible to do this, but may not be desireable to do this: + Right now it assumes user is filtering by name which is useful and easy in some ways + if it was by ID that's pretty specific and not necessarily better for the user + it's better that they can select by click + but it's limiting in that they can't type "Ford*" and get all items that start with "Ford " i.e. all Ford dealers + but if they want a specific ford dealer they can type "Ford Nashville Center" or whatever and get that specfic one +possible implementation: + it appears that the server code filter criteria builder is built with this in mind (see below) + so it may be a case of changing DataList field definitions like this one: + + FieldDefinitions.Add(new AyaDataListFieldDefinition + { + TKey = "WidgetName", + FieldKey = "widgetname", + AyaObjectType = (int)AyaType.Widget, + UiFieldDataType = (int)UiFieldDataType.Text, + SqlIdColumnName = "awidget.id", + SqlValueColumnName = "awidget.name", + IsRowId = true + }); + + to uiFieldDataType = InternalId instead and some fixup at the client datagrid to support that as a text view column. + For now I'm going to leave it as is, it's not the end of the world to have to redefine it (though post release it might get ugly) + +Server:datalistsqlcriteriafilterbuilder: + //prep for possible ID field filter instead + string columnNameToFilter = string.Empty; + UiFieldDataType DataTypeToFilter = UiFieldDataType.NoType; + //Check if filtering by ID rather than by name + //this is indicated by this column being an id type + //and by the comparison operator being eq or neq + //and the value being a number not a string + if (DataListField.HasIdColumn() && IsPossibleIdValue && (opType == DataListFilterComparisonOperator.Equality || opType == DataListFilterComparisonOperator.NotEqual)) + { + columnNameToFilter = DataListField.SqlIdColumnName; + DataTypeToFilter = UiFieldDataType.InternalId; + } + else + { + DataTypeToFilter = (UiFieldDataType)dataType; + columnNameToFilter = DataListField.GetSqlValueColumnName(); + } + + todo: VUE / VUETIFY ROADMAP RELEASES (Sept 23 2020 was put on hold as it appears vuetify is way behind so looking like post release but check again here) Q1 2021 is vuetify v3 https://vuetifyjs.com/en/introduction/roadmap/ diff --git a/ayanova/devdocs/todo.txt b/ayanova/devdocs/todo.txt index c6ecbe71..14612a51 100644 --- a/ayanova/devdocs/todo.txt +++ b/ayanova/devdocs/todo.txt @@ -6,12 +6,8 @@ MISC ITEMS THAT CAME UP - - -todo: missing feature, can't filter widgetlist by User due to no UserList being available - Is this meant to be text only filter and we have no id filter system? - How hard to add proper list selection and ID because it seems important? - fucking grids are always fucking fucked due to the filter and sort fucking shit +todo: test filtering linked objects in grids by name to see if that is sufficient for user purposes + e.g. widgetdatalist filter the User field by the name and see if that works well enough todo: tag search is unguessable without reading docs, could there be a hint or something? FAQ of the day? @@ -24,9 +20,7 @@ todo: User list selection checkbox is for...? remove it if there isn't a purpose to it and bulk ops are not supported - - -todo: TEST ON SERVER CUSTOMER STUFF +todo: TEST DEPLOY TO DEVOPS FOR TESTING NEW CUSTOMER STUFF THOROUGHLY ............................................................ diff --git a/ayanova/src/views/ay-data-list-view.vue b/ayanova/src/views/ay-data-list-view.vue index d270fac6..7c7c1c1c 100644 --- a/ayanova/src/views/ay-data-list-view.vue +++ b/ayanova/src/views/ay-data-list-view.vue @@ -42,6 +42,7 @@ + {{ item }}