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

Simplify the Discharge View by Converting it to a Serializer #1205

Closed
aeswibon opened this issue Feb 26, 2023 · 2 comments · Fixed by #1252
Closed

Simplify the Discharge View by Converting it to a Serializer #1205

aeswibon opened this issue Feb 26, 2023 · 2 comments · Fixed by #1252

Comments

@aeswibon
Copy link
Member

Feature Request

Is your feature request related to a problem? Please describe

  • The discharge view has become complicated, making it difficult to understand and manage.

Describe the solution you'd like

  • To simplify the discharge view and improve its functionality, it is suggested that the view be converted to a serializer. A serializer can effectively handle complex data structures and relationships, making the view more organized and user-friendly.

Describe alternatives you've considered

No alternative solutions were considered.

Additional context

This feature request was originally posted by user @vigneshhari in #1155 (comment)

Can you create another issue to move the discharge view to use a serializer? it was a rather simple view, now that it's complicated, I recommend using a serializer.

@aeswibon aeswibon self-assigned this Feb 26, 2023
@nihal467
Copy link
Member

@cp-coder

ohcnetwork/care_fe#5271
ohcnetwork/care_fe#5272

I am unassigning you from this backend issue since there are two issues you have already assigned last week in the care_fe and haven't pushed the fix yet. Once you push the fix for an already assigned issue, assign yourself new ones

@sainak sainak linked a pull request Apr 18, 2023 that will close this issue
3 tasks
@sainak
Copy link
Member

sainak commented May 20, 2023

closed by #1252

@sainak sainak closed this as completed May 20, 2023
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

Successfully merging a pull request may close this issue.

3 participants