Skip to content

UI server, memperbarui script dan update hal kecil lainnya #171

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 75 commits into from
Nov 19, 2021

Conversation

reacto11mecha
Copy link
Collaborator

@reacto11mecha reacto11mecha commented Oct 25, 2021

Closes #164

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

Testing

  • doctesting js
  • lint testing eslint

@reacto11mecha reacto11mecha requested a review from a team October 25, 2021 07:11
@reacto11mecha reacto11mecha self-assigned this Oct 25, 2021
@reacto11mecha reacto11mecha added the enhancement New feature or request label Oct 25, 2021
@reacto11mecha
Copy link
Collaborator Author

image

Tampilan saat ini

@reacto11mecha reacto11mecha mentioned this pull request Oct 25, 2021
@reacto11mecha
Copy link
Collaborator Author

Yang mau nyoba branch ini, caranya kayak begini.
Tambahin remotenya dulu, terserah mau yang https atau ssh

# https
git remote add ezra https://github.com/reacto11mecha/Javascript.git

# ssh
git remote add ezra git@github.com:reacto11mecha/Javascript.git

Baru track branchnya

git fetch ezra
git checkout --track ezra/ui-server

Kalo semisal aku buat perubahan, tinggal pull aja di branch yang ui-server

git pull

@azqilana
Copy link
Contributor

ditunggu dan akan di coba

@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2021

This pull request introduces 4 alerts when merging 717a8ba into d329ba7 - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression

@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2021

This pull request introduces 4 alerts when merging 0743fe6 into 2f4e170 - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression

@reacto11mecha
Copy link
Collaborator Author

Update, sekarang dah bisa nampilin folder sama file didalem foldernya

image

image

image

Untuk file readme.md emang sengaja disembunyiin karena readmenya udah ada langsung di folder ditempat dia berada. Untuk runner file js only sama html belum di implementasi dulu

@lgtm-com
Copy link

lgtm-com bot commented Oct 26, 2021

This pull request introduces 8 alerts when merging b3871bc into 2f4e170 - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression
  • 4 for Unused variable, import, function or class

@azqilana
Copy link
Contributor

azqilana commented Oct 26, 2021

Update, sekarang dah bisa nampilin folder sama file didalem foldernya

image

image

image

Untuk file readme.md emang sengaja disembunyiin karena readmenya udah ada langsung di folder ditempat dia berada. Untuk runner file js only sama html belum di implementasi dulu

gg untuk kerja kerasnya om @reacto11mecha

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2021

This pull request introduces 4 alerts when merging d153a81 into 8aa97dc - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2021

This pull request introduces 4 alerts when merging 6a5062a into 571f373 - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2021

This pull request introduces 4 alerts when merging db49cd9 into 814313a - view on LGTM.com

new alerts:

  • 4 for Uncontrolled data used in path expression

Copy link
Collaborator

@hasheemi hasheemi left a comment

Choose a reason for hiding this comment

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

Sip kalau gitu kayak nya udah semua

@nomadkode
Copy link
Contributor

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

@reacto11mecha
Copy link
Collaborator Author

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Okey, sekalian sama yang ini

@reacto11mecha
Copy link
Collaborator Author

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Selesai om, silahkan di review

@nomadkode
Copy link
Contributor

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Selesai om, silahkan di review

Udah oke sih harusnya. Cuma masih error. Help @slowy07

@reacto11mecha
Copy link
Collaborator Author

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Selesai om, silahkan di review

Udah oke sih harusnya. Cuma masih error. Help @slowy07

Kalo buat rule aman sih harusnya, dia perlu di merge dulu biar bisa kepake. Yang masalah aku ga ngerti di bagian node ci, kenapa dia failed

@nomadkode
Copy link
Contributor

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Selesai om, silahkan di review

Udah oke sih harusnya. Cuma masih error. Help @slowy07

Kalo buat rule aman sih harusnya, dia perlu di merge dulu biar bisa kepake. Yang masalah aku ga ngerti di bagian node ci, kenapa dia failed

Yaudah merge aja zra, kalo masih error bisa di solve nanti.

@reacto11mecha
Copy link
Collaborator Author

yaml nya bagian label perlu di edit ada bot mergefy yg error karena sudah gak ada tim docs.

Selesai om, silahkan di review

Udah oke sih harusnya. Cuma masih error. Help @slowy07

Kalo buat rule aman sih harusnya, dia perlu di merge dulu biar bisa kepake. Yang masalah aku ga ngerti di bagian node ci, kenapa dia failed

Yaudah merge aja zra, kalo masih error bisa di solve nanti.

Approve dulu om, terus kasih label. Dah aman kan docs nya

@dhafitf
Copy link
Member

dhafitf commented Nov 18, 2021

Waduh baru aja nemu yang salah om.
Bagian link yang mengarah ke folder, masih ada yang salah. Lengkapnya sudah saya kasih tau di discord.

@reacto11mecha
Copy link
Collaborator Author

Node CI nya failed jadi walaupun di kasih label ready-to-merge ga bakalan bisa. Mesti di merge manual

@nomadkode
Copy link
Contributor

Node CI nya failed jadi walaupun di kasih label ready-to-merge ga bakalan bisa. Mesti di merge manual

ngebuildnya masih failed yaa brarti?

@reacto11mecha
Copy link
Collaborator Author

Node CI nya failed jadi walaupun di kasih label ready-to-merge ga bakalan bisa. Mesti di merge manual

ngebuildnya masih failed yaa brarti?

Hooh

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.

baru ada waktu buat nyobain ini tadi, LGTM om

@nomadkode
Copy link
Contributor

Node CI nya failed jadi walaupun di kasih label ready-to-merge ga bakalan bisa. Mesti di merge manual

ngebuildnya masih failed yaa brarti?

Hooh

Jadi mau di merge dulu apa solve dulu nih masalah node CI nya? Kan bisa force merge pake manual kan.

@reacto11mecha
Copy link
Collaborator Author

Node CI nya failed jadi walaupun di kasih label ready-to-merge ga bakalan bisa. Mesti di merge manual

ngebuildnya masih failed yaa brarti?

Hooh

Jadi mau di merge dulu apa solve dulu nih masalah node CI nya? Kan bisa force merge pake manual kan.

Merge aja dulu, bisa di solve nanti

@reacto11mecha reacto11mecha merged commit 92a5488 into bellshade:main Nov 19, 2021
@reacto11mecha reacto11mecha deleted the ui-server branch November 19, 2021 01:16
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.

UI Server
7 participants