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

[2.x] - Upgrade ke laravel v6.x dan optimasi code #35

Merged
merged 15 commits into from
Nov 21, 2020

Conversation

agungsugiarto
Copy link
Member

Dalam PR ini dilakukan upgrade ke versi laravel 6.x

  • Coding standard menggunakan laminas coding standard
  • Optimasi code agar kompatibel dengan laravel 6.x
  • Optimasi packages
  • Menghapus folder vendor
  • Menambahkan packages developer debugbar untuk developer

@eddieridwan
Copy link
Contributor

@agungsugiarto maaf terlambat. Mudah2an bisa lihat minggu depan.

Copy link
Contributor

@eddieridwan eddieridwan left a comment

Choose a reason for hiding this comment

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

@agungsugiarto terima kasih.

Saya tidak bisa jalankan setelah folder vendor dihapus di PR ini. Apakah perlu ada perubahan di file index.php juga?

opendk_local_dashboard_profil

@agungsugiarto
Copy link
Member Author

agungsugiarto commented Nov 12, 2020

Pak @eddieridwan untuk laravel dia mengakses ke path public/index.php bukan di root/index.php. Bisa dicoba lewat command php artisan serve dahulu apakah error juga. Pastikan juga sebelum menjalankan lakukan command composer dump-autoload

@eddieridwan
Copy link
Contributor

Pak @eddieridwan untuk laravel dia mengakses ke path public/index.php bukan di root/index.php. Bisa dicoba lewat command php artisan serve dahulu apakah error juga. Pastikan juga sebelum menjalankan lakukan command composer dump-autoload

Error itu di public/index.php (https://github.com/OpenSID/OpenDK/blob/master/public/index.php). Apakah dengan menjalankan composer dump-autoload, index.php akan otomatis disesuaikan?

@agungsugiarto
Copy link
Member Author

agungsugiarto commented Nov 12, 2020

Error itu di public/index.php (https://github.com/OpenSID/OpenDK/blob/master/public/index.php). Apakah dengan menjalankan composer dump-autoload, index.php akan otomatis disesuaikan?

Saya kira ini hanya masalah teknis, apakah sudah menjalakan command composer install pak sebelumnya? composer dump-autoload hanya untuk menggenerate class map di vendor.

opendk_local_dashboard_profil

error seperti gambar diatas karena folder vendor belum ada, jadi harus menjalakan command composer install terlebih dahulu.

@eddieridwan
Copy link
Contributor

@agungsugiarto , jadi karena folder vendor tidak masuk repo, berarti harus bisa jalankan composer install di shared hosting (cPanel) setiap kali install dan update. Begitu? Umumnya, hosting yg digunakan oleh kecamatan adalah di shared hosting.

Mohon penjelasan bagaimana proses install dan update di cPanel setelah PR ini digabung ke master. Sekarang langkah yg diikuti seperti di https://github.com/OpenSID/OpenDK/wiki/Install-di-cPanel dan https://github.com/OpenSID/OpenDK/wiki/Update-di-cPanel.

Mohon bantuan anda untuk menyiapkan halaman wiki untuk install/update di cPanel setelah PR ini digabung. Terima kasih.

@eddieridwan
Copy link
Contributor

eddieridwan commented Nov 12, 2020

@agungsugiarto mungkin anda bisa bantu praktekkan dengan mengupdate demo OpenDK di https://demo.opendk.my.id. Kalau bersedia, nanti kami kirim login ke cPanelnya -- dan kalau bersedia akan bagus sekali kalau anda bisa tunjukkan menggunakan zoom kepada teman2 lain yg belum begitu paham dengan pemasangan aplikasi Laravel.

@agungsugiarto
Copy link
Member Author

Maaf pak @eddieridwan untuk urusan dokumentasi dan praktik di cPanel sebaiknya menunggu kontributor lain untuk ini.

@eddieridwan
Copy link
Contributor

Bung @agungsugiarto , saya sudah add anda ke channel #opendk di Slack (https://opensid.slack.com/archives/C01EGSTCWGJ). Mudah2an anda bersedia membantu memberi arahan dan penjelasan pada teman2 (termasuk saya) yg masih belajar Laravel dan meraba-raba cara mengembangkan OpenDK. Terima kasih sebelumnya.

@roaddee
Copy link
Contributor

roaddee commented Nov 16, 2020

saat ini sudah ada cara untuk jalankan composer install/update dari shared hosting pak @eddieridwan dengan syarat ada fasilitas terminal nya, saya cari di google sdh banyak, bahkan ada hosting yang sdh menyediakan composer dan npm sebagai fitur standar.

@agungsugiarto
Copy link
Member Author

Pak @roaddee mungkin bisa membantu untuk mengupdate dokumentasi di wiki?

@eddieridwan
Copy link
Contributor

Bung @agungsugiarto , mohon lihat pertanyaan di https://opensid.slack.com/archives/C01EGSTCWGJ/p1605863576000900. Terima kasih.

@eddieridwan eddieridwan merged commit 8a8ed84 into OpenSID:master Nov 21, 2020
@eddieridwan
Copy link
Contributor

@agungsugiarto terima kasih. Sdh digabung ke master, dengan perubahan:

  1. Commit composer.lock ke repo, supaya pengguna bisa menghasilkan folder vendor sesuai dengan yg telah diuji.
  2. Hapus yarn.lock, karena sekarang menggunakan composer.lock, dan bukan yarn.

Wiki perlu disesuaikan.

@roaddee
Copy link
Contributor

roaddee commented Nov 22, 2020

Pak @roaddee mungkin bisa membantu untuk mengupdate dokumentasi di wiki?

baik, saya sudah ujicoba di php 7.4 dan bisa sukses instalasi nya, tinggal mengujicoba saja semua fitur

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.

3 participants