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

Restrukturisasi Informasi API #373

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

vzrenggamani
Copy link
Contributor

@vzrenggamani vzrenggamani commented Aug 4, 2024

Selayang Pandang

Sepertinya sudah waktunya untuk merapikan daftar API yang ada di sini sekaligus memberikan "informasi awal" yang lebih jelas kepada para calon pengguna API melalui data tambahan yang ada.

Changes

  • Menghapus kolom status. Kolom tersebut tidak relevan karena kebanyakan orang menginput repository atau landing page. Sehingga status check akan 200 OK
  • Mengubah kolom autentikasi menjadi tags sesuai dengan keadaan API yang ada.
  • Menghapus seluruh API yang nonaktif.

Needs To Do

  • Migrasi penggunaan Bun dengan tools yang lebih umum digunakan will do in another PRs
  • Mengubah template PR dan Issues agar format yang diberikan kontributor lebih konsisten
  • Membuat GH Actions untuk memproses issues atau PR secara otomatis

References & Post Note

Closing #217

@sooluh
Copy link
Collaborator

sooluh commented Aug 5, 2024

Wah mantap mas, boleh dilanjutkan, nanti saya bantu review dan merge InsyaAllah.

@sooluh sooluh marked this pull request as draft August 5, 2024 09:44
@sooluh sooluh self-requested a review August 5, 2024 09:45
@vzrenggamani
Copy link
Contributor Author

Wah mantap mas, boleh dilanjutkan, nanti saya bantu review dan merge InsyaAllah.

Saya sebenarnya ingin mencoba cara mas untuk otomatisasi pengecekan status API hingga content generation, namun timbul dualiasme data source, antara README dengan JSON.

Andai ada DB seperti Google Spreadsheet yang bisa dimaintain secara publik gtu ya wkwk, kalo JSON sebenarnya asik tinggal flag "INACTIVE"..

@sooluh
Copy link
Collaborator

sooluh commented Aug 6, 2024

Andai ada DB seperti Google Spreadsheet yang bisa dimaintain secara publik

Kalo ini bisa aja sih mas, bikin Google Spreadsheet terus share public sebagai Editor, nanti mekanisme update ke repo-nya bisa pake GitHub action terjadwal per 24 jam mungkin buat hint ke API, atau bisa juga ada satu endpoint untuk Google Spreadsheet mengirim Webhook. Kita bisa memanfaatkan Google Apps Script.

Tapi, vibes kontribusi di GitHub-nya ngga kerasa mas haha

@vzrenggamani
Copy link
Contributor Author

vzrenggamani commented Aug 7, 2024

Tapi, vibes kontribusi di GitHub-nya ngga kerasa mas haha

Nah itu, sementara waktunya bersih - bersih API yg deprecated / inactive hihihi.

Kalo kayak gini, baiknya dibuat epics sekalian aja ya mas? kurang bagus kalo satu PR jadi MAJOR wkwkw

@vzrenggamani vzrenggamani marked this pull request as ready for review August 8, 2024 00:00
utils/remove_status_columns.py Outdated Show resolved Hide resolved
.github/workflows/review-issues.yml Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE/tambah_api.md Outdated Show resolved Hide resolved
.github/PULL_REQUEST_TEMPLATE/tambah_api.md Outdated Show resolved Hide resolved
.github/ISSUE_TEMPLATE/add_new_api.yml Outdated Show resolved Hide resolved
@sooluh
Copy link
Collaborator

sooluh commented Aug 11, 2024

Sebenernya beberapa ada yg API Wrapper ya mas, bukan RESTful API, pas awal-awal repository ini dibuat sama kang Fariz, kalo ngga salah list-nya semua RESTful API dan ready to use, jadi yg API Wrapper masih bisa masuk ke list repo ini ngga ya?

@vzrenggamani
Copy link
Contributor Author

Sebenernya beberapa ada yg API Wrapper ya mas, bukan RESTful API, pas awal-awal repository ini dibuat sama kang Fariz, kalo ngga salah list-nya semua RESTful API dan ready to use, jadi yg API Wrapper masih bisa masuk ke list repo ini ngga ya?

API wrapper ini maksudnya yang tools atau seperti wrapper API PDDIKTI yang ambil dari endpoint resmi PDDIKTI mas? jika tools, saya setuju untuk dibuang, jika untuk API wrapper dari endpoint resmi, masih "so-so" dikatakan API, hemat saya, selama informasi akurat, baiknya diapresiasi hehe

@sooluh
Copy link
Collaborator

sooluh commented Aug 17, 2024

Sebenernya beberapa ada yg API Wrapper ya mas, bukan RESTful API, pas awal-awal repository ini dibuat sama kang Fariz, kalo ngga salah list-nya semua RESTful API dan ready to use, jadi yg API Wrapper masih bisa masuk ke list repo ini ngga ya?

API wrapper ini maksudnya yang tools atau seperti wrapper API PDDIKTI yang ambil dari endpoint resmi PDDIKTI mas? jika tools, saya setuju untuk dibuang, jika untuk API wrapper dari endpoint resmi, masih "so-so" dikatakan API, hemat saya, selama informasi akurat, baiknya diapresiasi hehe

iya mas, yang API SDK/Wrapper, masih bisa dikatakan API ya? paling ditambahkan tags SDK aja, tapi nanti kedepannya gimana, barangkali jadi banyak SDK/Wrapper yang masuk ke sini, yang awalnya mau nyari RESTful API, malah ke-distract sama API SDK.

Copy link
Contributor Author

@vzrenggamani vzrenggamani left a comment

Choose a reason for hiding this comment

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

Added No-NSFW content declarations for the add new api request forms, both PRs and Issues.

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.

2 participants