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

Chart of accounts is expanded to allow for contribution related accounting categories #7207

Closed
Tracked by #7197
hdiniz opened this issue Jan 9, 2024 · 8 comments · Fixed by opencollective/opencollective-rest#531
Assignees
Labels
accounting All things asked by/for accountants api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) frontend

Comments

@hdiniz
Copy link

hdiniz commented Jan 9, 2024

Part of #7197

The chart-of-accounts MVP was created to support only expense-related accounting categories and now needs to be extended to cover contribution-related accounting-categories.

@hdiniz hdiniz added accounting All things asked by/for accountants api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) frontend labels Jan 9, 2024
@Betree
Copy link
Member

Betree commented Jan 16, 2024

API

  • Add AccountingCategoryId to Orders
  • Accept accountingCategory on addFunds mutation
  • Accept accountingCategory on createPendingOrder mutation
  • Accept accountingCategory on editPendingOrder mutation
  • Expose accountingCategory on Order object
  • Add kind on AccountingCategory with a limited subset: CONTRIBUTION, EXPENSE, ADDED_FUNDS
  • Add a kind argument on the host.accountingCategories query
  • Make sure contributions accounting categories cannot be used for expenses, & vis versa
  • Add a mutation to update the category of an existing order

Frontend

  • For the expenses queries, the host.accountingCategories query, add a filter to get only the expenses-related ones
  • Consolidate the accounting category checkbox in the export CSV modal to include orders

Rest

  • Fetch accounting category for orders

@shannondwray
Copy link

I see that the 'i don't know' option is still available for the contribution category! Is this needed as it's just fiscal host admins who would be involved in contribution categories?

@Betree
Copy link
Member

Betree commented Feb 22, 2024

@shannondwray we don't want to force expense submitters and collective admins to pick a category. In some cases, choosing the right one is challenging and it's better to have them pick "I don't know" rather than a random category.

@shannondwray
Copy link

okay, but when I was trying to submit an expense to OCF the i don't know doesn't show up. (I am probably a fiscal host admin user) and then when I go to add funds to a collective on one of my test accounts the I don't know option shows up. Which I don't think is necessary as contribution categorization is only for fiscal host admins. Does that make sense?

@Betree
Copy link
Member

Betree commented Feb 22, 2024

okay, but when I was trying to submit an expense to OCF the i don't know doesn't show up. (I am probably a fiscal host admin user)

Yes, it's because you're a host admin and host admins are not allowed to pick the "I don't know" option. We're currently reviewing this behavior. cc @iamronen.

and then when I go to add funds to a collective on one of my test accounts the I don't know option shows up

We're indeed lacking consistency here. I'll wait for @iamronen to pick the best approach.

@iamronen
Copy link
Contributor

  1. Yes, this is on my plate.
  2. @shannondwray these choices came from specific needs in specific contexts and they are indeed not consistent ... and they are evolving.
  3. @Betree as I was looking to recreate this I found that, for OSC (both in production and staging), the control for selecting accounting categories does not show up in the pending contributions and added funds flows? Why is that?

@shannondwray
Copy link

I believe that would be because they don't have any contribution categories activated

@Betree
Copy link
Member

Betree commented Mar 25, 2024

3. @Betree as I was looking to recreate this I found that, for OSC (both in production and staging), the control for selecting accounting categories does not show up in the pending contributions and added funds flows? Why is that?

@iamronen That must have been a feature flag or release issue, the category select does seem to appear:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accounting All things asked by/for accountants api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) frontend
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants