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

Enhancement: Kelas & Jurusan Module #130

Merged
merged 6 commits into from
Aug 1, 2024

Conversation

reactmore
Copy link
Contributor

@reactmore reactmore commented Jul 30, 2024

Deskripsi

Mengganti resourceController ke baseController, Implementasi validasi dari helper sebelumnya dibuat, serta menambahkan animasi spinner dan recode javascript code.

Masalah Terkait

Perubahan yang Diajukan

  • Update Controller JurusanController & KelasController
  • Update Model JurusanModel & KelasModel
  • Update Views & Css
  • Penambahan Plugin Sweetalert
  • Penambahan Custom File Javascript
  • Update Route

Checklist

Pastikan semua item berikut telah Anda periksa sebelum mengirimkan permintaan pull:

  • Perubahan telah diuji dan berfungsi dengan baik.
  • Dokumentasi telah diperbarui sesuai dengan perubahan.
  • Tidak ada konflik merge dengan cabang utama.

Catatan Tambahan

Code Sweetalert untuk delete data bisa digunakan berulang

<button onclick='deleteItem("admin/kelas/deleteKelasPost","<?= $value["id_kelas"]; ?>","Konfirmasi untuk menghapus data");' class="btn btn-danger p-2" id="<?= $value['id_kelas']; ?>">
              <i class="material-icons">delete_forever</i>
              Delete
            </button>

deleteItem(url, id, text); 

Controller:
ex: 144

public function deleteKelasPost($id = null)
    {
        $id = inputPost('id');
        $kelas = $this->kelasModel->getKelas($id);
// Check data ada atau tidak
        if (!empty($kelas)) {
// Jika ada validasi lain tambahkan disini 
            if ($this->kelasModel->deleteKelas($id)) {
                $this->session->setFlashdata('success', 'Data berhasil dihapus');
            } else {
                $this->session->setFlashdata('error', 'Gagal menghapus data');
            }
        }
    }
}
Screen.Recording.2024-07-30.115447.mp4

@reactmore reactmore marked this pull request as ready for review July 30, 2024 05:25
@nanda443
Copy link
Contributor

delete kelas dengan id=1 tidak bisa.

screenshoot

Screenshot 2024-07-30 162543

@reactmore
Copy link
Contributor Author

delete kelas dengan id=1 tidak bisa.

screenshoot Screenshot 2024-07-30 162543

Baik Ternyata di database ada fk ke tb siswa nanti saya tambah kondisinya lagi seperti jurusan

@nanda443
Copy link
Contributor

Sepertinya karena di lokal saya ada data siswa dengan kelas tersebut kak😅 mohon maaf ya🙏

@reactmore
Copy link
Contributor Author

Sepertinya karena di lokal saya ada data siswa dengan kelas tersebut kak😅 mohon maaf ya🙏

Tambah Flashdata Error untuk Kelas yang punya relasi dengan tb siswa
image

@ikhsan3adi ikhsan3adi added the enhancement New feature or request label Jul 30, 2024
@ikhsan3adi ikhsan3adi merged commit 9ddbfb8 into ikhsan3adi:master Aug 1, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants