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.

Sign up for GitHub

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

Jump to bottom

(refactor) Inventaire en Stock #1847

Closed
DedrickEnc opened this issue Jul 12, 2017 · 4 comments
Closed

(refactor) Inventaire en Stock #1847

DedrickEnc opened this issue Jul 12, 2017 · 4 comments
Labels
Needs Discussion Refactor

Comments

@DedrickEnc
Copy link
Contributor

The inventaire en stock module is on master, here is some noticed points when using it :

  1. the record is grouped by depot, is it a way to remove depot name in the record item?

  2. Inventaire table should be refactored refactor: Invoice registry #1749

  3. The total as aggregate is not important since it is semantically incorrect or not meaningful.

  4. columns CM and SS must be more clear let's choose other name

  5. What is the meaning of the last column?
@jniles
Copy link
Collaborator

jniles commented Oct 4, 2017

@DedrickEnc @mbayopanda - I know @WayneNiles wants to begin testing this in-house at IMA World Health. If this registry is still confusing, we should come to a decision about it and clean it up.

@DedrickEnc
Copy link
Contributor Author

@jniles
It seems like, it is already cleaned.

@jniles
Copy link
Collaborator

jniles commented Oct 5, 2017

Great! Can this issue be closed yet, or are there still points to address?

@DedrickEnc
Copy link
Contributor Author

It should be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Needs Discussion Refactor
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jniles @DedrickEnc