Skip to content

docs: menambahkan materi ke 2 OOP dan memberbaiki typo #260

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

Merged
merged 9 commits into from
Jan 24, 2022

Conversation

hasheemi
Copy link
Collaborator

Closes #221

Deskripsi (Description)

  • saya menambahkan algoritma terbaru
  • saya memperbaiki algoritma yang sudah ada
  • saya menambahkan materi/sub materi belajar
  • saya memperbaiki materi/sub materi belajar
  • saya menambah dokumentasi
  • saya memperbaiki dokumentasi

Contributor Requirements (Syarat Kontributor)

  • saya sudah membaca (i have read) CONTRIBUTING dan sudah menyetujui semua
  • saya telah menambahkan docstring yang memberikan penjelasan maksud dari kode yang saya buat
  • saya menggunakan bahasa indonesia untuk memberikan penjelasan dari kode yang saya buat

Environment

saya menggunakan (im used):

  • os = linux / windows / macOS

Testing

  • doctesting js
  • lint testing eslint

@hasheemi hasheemi requested a review from wahyall January 23, 2022 07:17
Copy link
Collaborator

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Nama filenya sekalian diubah dong jadi semisal example.js biar ga bias maka filenya.

Copy link
Collaborator

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Oiya lupa, di readmennya juga arahin ke filenya biar bisa langsung di klik dari dokumentasinya

Copy link
Collaborator

@reacto11mecha reacto11mecha left a comment

Choose a reason for hiding this comment

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

Sip, dah oke bagiku, LGTM.

Request review dokumentasi @dhafitf

@reacto11mecha reacto11mecha requested a review from a team January 24, 2022 08:24
Copy link
Collaborator

@rizkyikhwan rizkyikhwan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@rolandowebdev rolandowebdev left a comment

Choose a reason for hiding this comment

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

Sedikit saran perubahan

Yang diubah pertama :

  • suatu => sebuah
  • Javascript => javascript
  • Array juga termasuk object. => Perlu diingat array juga termasuk sebuah object.

Object adalah sebuah tipe data di javascript yang bisa menyimpan banyak value atau nilai.
Sama seperti array, yang membedakan adalah array memiliki index sedangkan object memiliki property.
Perlu diingat array juga termasuk sebuah object.

Yang diubah kedua :

  • 14 => 20

const person = {
name: "salim",
age: 20 // karena value diobjectnya 20
};

name dan age disebut property sedangkan "salim" dan 20 disebut value.

@dhafitf dhafitf self-requested a review January 24, 2022 15:22
@dhafitf
Copy link
Member

dhafitf commented Jan 24, 2022

Sedikit saran perubahan

Yang diubah pertama :

  • suatu => sebuah
  • Javascript => javascript
  • Array juga termasuk object. => Perlu diingat array juga termasuk sebuah object.

Object adalah sebuah tipe data di javascript yang bisa menyimpan banyak value atau nilai. Sama seperti array, yang membedakan adalah array memiliki index sedangkan object memiliki property. Perlu diingat array juga termasuk sebuah object.

Yang diubah kedua :

  • 14 => 20

const person = { name: "salim", age: 20 // karena value diobjectnya 20 };

name dan age disebut property sedangkan "salim" dan 20 disebut value.

Oke terima kasih. Tadi saya kelewatan di value yang 14.
Untuk yang Javascript, saya rasa tidak masalah, karena jika saya melihat di sumber-sumber lain, penulisannya justru menggunakan JavaScript.

@dhafitf dhafitf requested a review from rolandowebdev January 24, 2022 15:36
Copy link
Collaborator

@rolandowebdev rolandowebdev left a comment

Choose a reason for hiding this comment

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

Ok udah aman...LGTM🚀

@mergify mergify bot merged commit a4057b5 into bellshade:main Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Object, property, and method
7 participants