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

feat: Title is not returned when getting list of Appointments via API. #7301

Closed
AngelsDustz opened this issue Mar 28, 2024 · 0 comments · Fixed by #7303
Closed

feat: Title is not returned when getting list of Appointments via API. #7301

AngelsDustz opened this issue Mar 28, 2024 · 0 comments · Fixed by #7303
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Milestone
7.0.2.1

Comments

@AngelsDustz
Copy link
Contributor

Describe the bug

When you get a list of all Appointments it does not return the title for said appointment making it impossible to create a proper list. The hometext however is returned.

To Reproduce

Steps to reproduce the behavior:

  1. Do a GET request to /api/appointment
  2. See no title field

Expected behavior

The title is returned for the list operation.

Client configuration

  • Browser: N.A.
  • OpenEMR version: 7.0.2
  • Operating system: N.A.

It seems it was intentionally not added? Which is kinda weird for now you need to do a request PER appointment to get the title.
AngelsDustz added a commit to AngelsDustz/openemr-fixes that referenced this issue Mar 28, 2024
@AngelsDustz
Resolve openemr#7301: Add title to appointment
62f1d5c
@AngelsDustz AngelsDustz mentioned this issue Mar 28, 2024
Merged
@adunsulag adunsulag changed the title [bug?] Title is not returned when getting list of Appointments via API. feat: Title is not returned when getting list of Appointments via API. Mar 28, 2024
adunsulag pushed a commit that referenced this issue Mar 29, 2024
@AngelsDustz
Resolve #7301: Add title to appointment (#7303)
ed58faf
sjpadgett added a commit to sjpadgett/openemr that referenced this issue Mar 30, 2024
@sjpadgett
Merge branch 'master' of https://github.com/openemr/openemr into weno…
7cd2457 
…_new

* 'master' of https://github.com/openemr/openemr:
  Resolve openemr#7300: Resolve bug accidentally introduced in API (openemr#7302)
  Resolve openemr#7301: Add title to appointment (openemr#7303)
  fix: missing attr _insurance_edit_screen_edit.html.twig (openemr#7298)
  Weno fixup and credential API (openemr#7256)
  fix: separate pt and enc billing note when saving invoice(openemr#7296)
  fix: add canceled appts counter to report (openemr#7295)
  fix: ERA posting based on code mod combo billed that prevents insert into billing table (openemr#7294)
  fix: patient select bug from billing manager
  fix: silence warning
stephenwaite pushed a commit to stephenwaite/openemr that referenced this issue Apr 6, 2024
@AngelsDustz @stephenwaite
Resolve openemr#7301: Add title to appointment (openemr#7303)
64e4d3d
@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 23, 2024
@adunsulag adunsulag added the developers This issue targets an issue that is for developers/collaborators/module writers/technical users label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
developers This issue targets an issue that is for developers/collaborators/module writers/technical users
Projects
None yet
Milestone
7.0.2.1
Development

Successfully merging a pull request may close this issue.

Resolve #7301: Add title to appointment AngelsDustz/openemr-fixes
2 participants
@adunsulag @AngelsDustz