phpindonesia.or.id-membership2 icon indicating copy to clipboard operation
phpindonesia.or.id-membership2 copied to clipboard

Published 20 hours ago • verified publisher icon phpindonesia

[WIP] Refactory & Enhancement

Open feryardiant opened this issue 7 years ago • 8 comments

Perubahan

  • [x] Tidak lagi menggunakan folder www sebagai document root & menambahkan folder asset untuk menampung semua file CSS, JS, Fonts & Images.
  • [x] Mengubah codebase, detail perubahan bisa dilihat di tiap commit message.
  • [x] Penggunaan file .env untuk setting dasar. Meminimalisir terjadinya konflik ketika ada perubahan atau penambahan key di file app/settings.php.

Perbaikan

  • [x] Memperbaiki mailer (terkait #94), disini mailer akan kita ubah dari sebelumnya menggunakan postmark (SMTP) ke sparkpost (ReST API) dimana kelebihan utama dari penggunaan ReST API adalah proses pengiriman email dibebankan ke provider, bukan server production. Selain itu sparkpost juga menawarkan fitur mail marketing secara gratis jadi jika nantinya kita butuh untuk bulk email dapat di-handle oleh sparkpost dengan lebih efisien.
  • [ ] Optimasi tampilan pada mobile (terkait #80)

Penambahan (Experimental)

  • [ ] Optimasi static asset (CSS, JS & Image) dengan tools. Opsi sementara adalah gulp atau webpack.
  • [ ] Menggunakan heroku sebagai staging sekaligus meng-enable fitur review, memungkinkan tiap pull request (nantinya) akan ada online demo yang di generate otomatis oleh heroku.
  • [ ] Menyediakan testing, walaupun tidak semuanya akan di test tapi setidaknya kita punya fitur ini untuk memastikan berjalan semestinya.

feryardiant avatar Apr 28 '17 18:04 feryardiant

Reviewing

okaprinarjaya avatar Apr 29 '17 02:04 okaprinarjaya

Thanks om @okaprinarjaya, ada ide apa aja yang perlu dibenahi lagi?

feryardiant avatar Apr 29 '17 14:04 feryardiant

Saya matur suwuunn bangett kepada om @feryardiant , sudah meluangkan waktunya. apakah ada ide lagi? saya pikir cukup, item-item di section Penambahan (Experimental) adalah ide-ide yg cukup besar dan harus digiring supaya terimplementasi.

Silahkan di-merge om, saya gak berani merge, karena om Fery yg lebih tau apakah PR ini sudah complete atau belom. Atau apakah ada yg butuh ditambahkan lg saya sementara belum tahu.

okaprinarjaya avatar Apr 29 '17 15:04 okaprinarjaya

Sami-sami om @okaprinarjaya, kebetulan mumpung saya ada waktu luang weekend ini jadi saya optimalkan untuk ngurusin repo ini (yang sudah saya terlantarkan cukup lama, hiks 😢).

Sementara PR ini masih berstatus WIP & masih ada TODO yang belum selesai tidak akan saya merge dulu sampai saya rasa sudah benar-benar siap. 🥂

feryardiant avatar Apr 29 '17 18:04 feryardiant

Menurut ane yg paling penting itu testing-nya dulu oom. Mungkin tepatnya functional atau acceptance testing-nya kali ya, kalo unit atau integration test kayanya rada mubazir kalo liat codebase yg sekarang.

Kalo boleh saran juga, sebaiknya bertahap, sedikit2 langsung​ di-pr n merge, dibanding nunggu banyak n langsung merge sekaligus.
Misal pr pertama ubah struktur direktori, selesai langsung merge.
Trus pr buat functional/acceptance testing, selesai langsung merge.

herloct avatar May 01 '17 02:05 herloct

Memperbaiki mailer (terkait #94)...

Yg ini ga langsung di-merge aja oom?

herloct avatar May 01 '17 05:05 herloct

Saya perlu memastikan semua todo disini sudah saya selesaikan dulu & maaf kalo mungkin akan sedikit makan waktu.

Saya juga masih ada kendala di salah satu penambahan (eksperimental) fitur yaitu penggunaan Heroku Review App yaitu heroku/heroku-buildpack-php#227

feryardiant avatar May 01 '17 10:05 feryardiant

Mailer done thru Sparkpost.

image

Tinggal bershi-bersih yang lain.. 🎉 🎉 🥂

feryardiant avatar May 04 '17 00:05 feryardiant