search

phpindonesia / phpindonesia.or.id-membership2

PHP Indonesia - Membership Application - Reloaded
http://membership.phpindonesia.or.id/
35 stars 98 forks source link

Re- #48 #51

Closed feryardiant closed 8 years ago

feryardiant commented 8 years ago

Cuma bantu mas @supalpuket untuk rebase #48 aja & belum saya test :sweat_smile:

okaprinarjaya commented 8 years ago

Mas @feryardiant , di pull request yg ini blom ada code yg pakai cloudinary untuk handling upload gambarnya. di branch master dan development sekarang sudah resmi pakai cloudinary

yus-ham commented 8 years ago

gak apa-apa biar saya saja bang :)

okaprinarjaya commented 8 years ago

Halooo mas @supalpuket , hahahahaha gimana niii.?? ni saya sedang on progress handle pull request nya mas Yusup. Saya pakai pull requestnya mas @feryardiant yg nge-rebase pull requestnya mas Yusup di #48 .

Mas Yusup yg mau lanjut untuk mengulang penambahan code-code nya dengan kondisi branch development yg baru saja ada banyaakkk perubahan?

okaprinarjaya commented 8 years ago

Waduuw.. :D mas @feryardiant , @supalpuket saya gak jadi merge deh perubahan-perubahan di #51 atau #48 ini indentasinya jadi 2 spasi dan alhasil brantakkan.. bingung mau resolvenya.

Tapi, perubahan code nya mas @supalpuket sangat bagus dan memang saya butuh improvement code itu. Klo mas Yusup berkenan, bolehkah dicoding ulang perubahan-perubahannya? dengan terlebih dahulu menge-pull-request ulang branch development yg terbaru saat ini.

Dan, saya minta tolong kpd mas Yusup, tolong pastikan improvement2 pada class Validator tidak mengganggu function2 yg lain. Improvement2 pada class Validator masih bisa dipakai di function2 yg lain. Mengingatkan saja, takutnya lupa :)

Terimakasih ^_^