search

cowinapi / developer.cowin

This group is created to facilitate technical and integration discussions related to cowin platform. API related contents can be obtained at API setu portal https://apisetu.gov.in/public/marketplace/api/cowin
115 stars 30 forks source link

string "ssssss" printed to console on loadCentersByAvailableData() #201

Open karanssh opened 3 years ago

karanssh commented 3 years ago

Hi Cowin devs, I observe that sssss is printed to console every time I search for vaccine slot from the cowin website. Found this code snippet:

loadCentersByAvailableData(t) { t || (console.log("ssssss"), this.appointmentForm.get("minAgeFilter").setValue(""), this.appointmentForm.get("maxAgeFilter").setValue(""), this.appointmentForm.get("covaxinFilter").setValue(""), this.appointmentForm.get("covishieldFilter").setValue(""), this.appointmentForm.get("freeFilter").setValue(""), this.appointmentForm.get("paidFilter").setValue("")); let n One more I see: this.mobileOtpForm.valid) { const t = this.mobileForm.get("mobile_number").value , e = this.encDecService.setSha256(this.mobileOtpForm.get("otp").value); let o = this.encDecService.set(JSON.stringify(t)) , i = { otp: e, txnId: this.txnNo }; this.disableVerifyBtn = !0, setTimeout(()=>{ console.log("dissss"), this.disableVerifyBtn = !1 } , 1e3), could you consider removing it? It's not really a big deal but it seems weird to have this in prod

ninjabhishek commented 3 years ago

somebody passed the code on prod with this :D

anshulbharadwaj commented 3 years ago

There are lot of console.log statements in the code. Code has been promoted to master(production) branch without minification