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

Menambahkan fitur edit dan hapus pada menu namagement dan sub menu management #3

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

Conversation

hamdanms
Copy link

halo pak nama saya muhammad hamdan ms, biasanya di panggil hamdan, saya mahasiswa semester akhir di Universitas Negeri Makassar (UNM), saya berasal dari kabupaten SINJAI, saya sangat terbantu dengan vidio tutorial CI yang bapak buat, dan berkat video itu sayapun jadi bisa belajar CI dengan mudah, dan sekarang saya mencoba mengikuti tutorial GIT dan GITHUB bapak dan sekarang saya sedang mencoba untuk :

  • FORK repo wp-login bapak
  • CLONE hasil repo yang telah saya fork
  • Mengupdate fitur Menu Management dan Sub Menu Management
  • Membuat Branch (di pc saya)
  • Merge Branch yang telah saya commit
  • Push master yang telah saya merge ke remote saya
  • Membuat Pull request dari repo hasil fork saya ke repo bapak

terimakasih sebanyak banyaknya pak atas ilmu yang telah bapak bagikan, sangat membantu bagi diri saya sendiri.

- memperbaiki tampilan
- menambahkan modal edit
- memperbaiki tombol edit dan delete
- memperbahatui link dan action

controllers/menu.php =
- menambahkan fungsi hapus dan edit
- menambahkan fungsi edit dan hapus

models/menu_model.php =
- menambahkan fungsi saveSubMenu
- menambahkan fungsi deleteSubMenu
@oxygenfox
Copy link

Ijin om saya pake cara ini untuk nambah edit dan delete cuman setiap di refresh data submenu tiba2 pindah tapi di cek di database urutannya normal ada pencrrahaan nya gk 😣

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