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

KFA Integration #38

Merged
merged 8 commits into from
Apr 6, 2024
Merged

Conversation

yudistirasd
Copy link
Collaborator

@yudistirasd yudistirasd commented Apr 6, 2024

#31

perlu disediakan wiki tidak pak @ivanwilliammd ?
jika iya, contributenya bagaimana?

ivanwilliammd and others added 8 commits April 3, 2024 11:41
- second parameter pada getenv tidak berfungsi sebagai default value, perlu ditambahkan handler ?: jika user tidak define variable di env
- define base url dan enpoint displit :
  - pada master data - KFA hanya menggunakan base url + endpoint
  - pada FHIR / Auth menggunakan base url + prefix ('oauth2/v1' | 'fhir-r4/v1') + endpoint
@ivanwilliammd
Copy link
Owner

#31

perlu disediakan wiki tidak pak @ivanwilliammd ? jika iya, contributenya bagaimana?

Boleh tlg disediakan Wiki-nya juga?
Caranya update dulu wiki-nya di tempat masing-masing nanti share linknya disini
Nanti akan saya tambahkan

Copy link
Owner

@ivanwilliammd ivanwilliammd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Untuk OAuthClient yg diganti apakah bisa diupdate juga di Wiki @yudistirasd ?

sekalian nanti dengan cara using package KFA...
Trims

@yudistirasd
Copy link
Collaborator Author

yudistirasd commented Apr 6, 2024 via email

@yudistirasd
Copy link
Collaborator Author

Untuk OAuthClient yg diganti apakah bisa diupdate juga di Wiki @yudistirasd ?

sekalian nanti dengan cara using package KFA... Trims

https://github.com/yudistirasd/satusehat-integration-wiki

@ivanwilliammd ivanwilliammd merged commit d42e3d0 into ivanwilliammd:31-api-kfa Apr 6, 2024
1 check passed
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 this pull request may close these issues.

None yet

2 participants