Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values #22

Open
ConnorGriffin opened this issue Jan 9, 2019 · 0 comments

Comments

@ConnorGriffin
Copy link
Owner

Currently all fields that are unique (Category, Budget, Account, Payee, etc.) require the user to provide the name rather than the ID. This was done to avoid hundreds of branching parameter sets (max is 32 I believe, but more than a few is unmanageable) based on allowing a mix of IDs and Names for all fields. For example adding a transaction could have 16 different parameter sets at a minimum (different combinations of: Budget, Account, Category, Payee).

Right now these functions will lookup all items for a parameter, and then get the ID by searching the list by name. Each of these parameters could be reconfigured to perform a regex check to find out if an ID was entered instead, and avoid the recursive name/ID lookups if the regex check passes.

All IDs follow this format: ^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$

  • Add a regex check, skip the name lookup if an ID is provided
  • Autocomplete should continue to use the names rather than IDs. Find out if there's a way to use the IDs while showing the user the names?
  • Piping outputs between functions should use the IDs instead of the names. May need to utilize custom formats for this.
@ConnorGriffin ConnorGriffin changed the title Allow use of ID fields (CategoryID, BudgedID, etc.) instead of forcing users to use the Names Allow optional use of ID fields (CategoryID, BudgedID, etc.) instead of forcing users to use the Names. Jan 9, 2019
@ConnorGriffin ConnorGriffin changed the title Allow optional use of ID fields (CategoryID, BudgedID, etc.) instead of forcing users to use the Names. Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use the Names. Jan 9, 2019
@ConnorGriffin ConnorGriffin changed the title Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use the Names. Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values. Jan 9, 2019
@ConnorGriffin ConnorGriffin changed the title Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values. Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values. Jan 9, 2019
@ConnorGriffin ConnorGriffin changed the title Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values. Allow optional use of ID fields (BudgetID, CategoryID, etc.) instead of forcing users to use name values Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant