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

add: menambahkan materi Decorators di TypescriptBasic #29

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

Mednoob
Copy link
Collaborator

@Mednoob Mednoob commented Mar 11, 2022

Deskripsi (Description)

Materi yang saya ingin tambahkan pada PR ini adalah materi Decorator.
Materi ini masih dalam tahap pengerjaan/draft
Hal-hal yang perlu diselesaikan:

  • mencari cara agar dapat dilakukan testing (Babel tidak dapat meng-compile decorator secara langsung)
  • menuliskan comment pada file materi (Decorators.ts)
  • menuliskan README

==============================================

  • saya menambahkan algoritma terbaru
  • saya memperbaiki algoritma yang sudah ada
  • saya memperbaiki dokumentasi
  • saya menambah dokumentasi

Contributor Requirements (Syarat Kontributor)

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

Environment

saya menggunakan (im used):

  • os = windows

Testing

  • lint testing eslint

Maintainer

maintainer typescript
@bellshade/typescript-team

@ikr4-m
Copy link
Collaborator

ikr4-m commented Mar 18, 2022

Untuk decorators saya baru sadar ternyata butuh dinyalakan di tsconfig.json, saya nyalakan dulu

@ikr4-m
Copy link
Collaborator

ikr4-m commented Mar 18, 2022

Sayangnya, untuk di Jest sendiri belum ada cara untuk mengecek apakah Decorator berfungsi apa tidak, tapi setidaknya bisa kita coba metode ini:
https://stackoverflow.com/questions/39112921/unit-testing-typescript-decorators

@ikr4-m ikr4-m self-assigned this Mar 18, 2022
@ikr4-m ikr4-m added bug Something isn't working help wanted Extra attention is needed labels Mar 18, 2022
@Mednoob
Copy link
Collaborator Author

Mednoob commented Mar 19, 2022

Setelah saya cari-cari di google, ternyata cara menggunakan @babel/plugin-proposal-decorators di file babel.config.js adalah dengan mengimport pluginnya (require('@babel/plugin-proposal-decorators').default), set option legacy ke true, dan ditaruh di property plugins

Img
(Abaikan garis merah, sepertinya TS di saya ngebug wkwkwk)

@ikr4-m
Copy link
Collaborator

ikr4-m commented Mar 24, 2022

Setelah saya cari-cari di google, ternyata cara menggunakan @babel/plugin-proposal-decorators di file babel.config.js adalah dengan mengimport pluginnya (require('@babel/plugin-proposal-decorators').default), set option legacy ke true, dan ditaruh di property plugins

Img (Abaikan garis merah, sepertinya TS di saya ngebug wkwkwk)

Untuk ini, saya sudah coba untuk melakukannya, namun sayangnya ngebug karena mesti import @babel/plugin-proposal-decorators padahal udah dimasukkan. Bisa dicek pada commit 4055fc5 dan di sini semisal ingin referensinya.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants