search

harrysenjaya / ScreenSaver

Skripsi
2 stars 2 forks source link

Basic version of SIAkadDataPuller, compile OK but not tested. #1

Closed pascalalfadian closed 3 years ago

pascalalfadian commented 3 years ago

Hi @harrysenjaya,

Ini saya kirimkan Pull Request, untuk menambahkan SIAkadDataPuller, seperti pada slide PPT saya yang terdahulu ( DataPuller).

Silakan direview, dan di accept kalau sudah oke ya.

Kalau sudah, tolong implementasi kamu disesuaikan seperti ini, jadi:

  1. Bikin package id.ac.unpar.screensaver.studentportal (atau id.ac.unpar.screensaver.pam) juga boleh.
  2. Bikinkan StudentPortalDataPuller seperti di PPT saya sebelumnya, di package itu.
  3. Kelas-kelas tambahan yang diperlukan untuk komunikasi, misalkan Scraper, pindahkan ke package tersebut.
  4. Di app yang kamu buat, dalam hal berkomunikasi hanya menggunakan StudentPortalDataPuller, tidak boleh kelas yang lain. Gunanya, supaya nanti saya bisa dengan mudah mengganti StudentPortalDataPuller ke SIAkadDataPuller.

Trims.

harrysenjaya commented 3 years ago

Baik Pa, nanti akan saya coba dan kabari progress dan hasilnya. Terima kasih.

harrysenjaya commented 3 years ago

Maaf baru mengabari lagi Pa. Apakah yang bapa maksud review itu melihat kodenya? Karena saya tidak bisa menjalankan aplikasinya dikarenakan tidak memiliki file login dosen propertiesnya.

pascalalfadian commented 3 years ago

Iya review kodenya aja. Tidak perlu dicoba. Trims.

harrysenjaya commented 3 years ago

Setelah saya cek, ternyata hampir seluruh atribut yang digunakan berbeda dengan versi studentportal, jadi saya kurang bisa melakukan pengecekan secara detil. Jadi saya accept saja ya Pa. Terima kasih.

pascalalfadian commented 3 years ago

Oke gapapa. Trims ya.