coronasafe/care_fe#2615

coronasafe/care#810

coronasafe/care_fe#2614

> @Ashesh3 @gigincg is it good for testing ?

Yep, it is!

coronasafe/care_fe#2525

Using `/\S+/` to check if the string is empty or not. 

![image](https://user-images.githubusercontent.com/3626859/170520528-94489786-c65c-40cc-857e-a2fe9ee0a507.png)



Now that's one smart way to hide your last seen! 👀 

coronasafe/care_fe#2535

Fix inconsistent type of visited countries attribute

coronasafe/care_fe#2536

coronasafe/care#789

> > I believe that the intention of displaying the message "Comment should contain at least 1 character" to the user is to prevent mishaps where they user might accidently try to hit the "Post comment"/"Add note" button without actually filling the box.
> > Sure, they can still bypass it by adding a space. But if we then add a check for the space, we would need to address dozens other ways where the user might try to post an empty comment. For example, using empty Unicode characters or even unintentional characters like a single dot or multiple spaces, tabs etc.
> > Although adding this is a neat little change, I think it might be out of scope of the behavior we are expecting from the check there currently is (Which is only checking length of the TextField's value).
> > TLDR; There would be many ways to bypass this check and it's main purpose should be limited to just nudging the user in case they hit submit without any text, accidently.
> > Just my thoughts, would love to hear other opinions on this! smile
> 
> Yeah @Ashesh3 I completely agree on this but comment without anything looks a little unprofessional it can contain anything a dot or any single character but containing space can make it look empty :) Even in whatsapp try to send a message containing only spaces no character it would not allow you to send it. Having this check maybe a bit enhancement 😄

Yep, agree with you there, could be a neat enhancement! 
We would also need to add a check for invisible Unicode characters too, since that would let users post a blank (but non empty) comment. 

Bonus: Try copy pasting this character `‏‏‎ ‎` (select the code block and copy), into Whatsapp and see what happens 😉 

coronasafe/care_fe#2532

Update pre-commit requirement and fix it's branch name

coronasafe/care#788

I believe that the intention of displaying the message "Comment should contain at least 1 character" to the user is to prevent mishaps where they user might accidently try to hit the "Post comment"/"Add note" button without actually filling the box.

Sure, they can still bypass it by adding a space. But if we then add a check for the space, we would need to address dozens other ways where the user might try to post an empty comment. For example, using empty Unicode characters or even unintentional characters like a single dot or multiple spaces, tabs etc. 

Although adding this is a neat little change, I think it might be out of scope of the behavior we are expecting from the check there currently is (Which is only checking length of the TextField's value).

TLDR; There would be many ways to bypass this check and it's main purpose should be limited to just nudging the user in case they hit submit without any text, accidently.

Just my thoughts, would love to hear other opinions on this! 😄 

Fix notification padding when no results are found

coronasafe/care_fe#2433

coronasafe/care_fe#2309

coronasafe/care_fe#2304

coronasafe/care_fe#2406

Empty notes and comments should not be allowed

coronasafe/care_fe#2507

UI issues on Upload External Results page

coronasafe/care_fe#2505

Improve Sample Test page and Voice Recorder

coronasafe/care_fe#2442

When the user denies the browser's microphone permission, The UI has no way of informing the user, instead it just continues recording. This needs to be fixed to inform the user about the same and not show "Recording" when there is no permission.

![image](https://user-images.githubusercontent.com/3626859/169289293-80a92721-cfeb-45f1-b65a-b782aa9f2ea8.png)

Shows Recording even though I have denied it microphone permission.

coronasafe/care_fe#2418

ICMR Sample export fails for new patients

coronasafe/care_fe#2440

Reduced the spacing

![image](https://user-images.githubusercontent.com/3626859/169153051-57a28d0a-aca9-4511-b726-c4a6ee6c17c4.png)


The above also applies to other parts of the same page:

![image](https://user-images.githubusercontent.com/3626859/169021973-68157161-a5db-40f5-b271-5c5f1f8e617c.png)


All the elements are now aligned to the line below the blood group.

![image](https://user-images.githubusercontent.com/3626859/168916108-5bb11745-920d-445b-b032-4ffe1d530738.png)


> @Ashesh3 tested on 15.4 works fine ![image](https://user-images.githubusercontent.com/42417893/168906614-ae6d7dde-7e58-47d9-ac39-29189eb4b4a9.png)

That's helpful! I only have versions <15.3 to test on, Guess it works with 15.4+ . I guess in either case the attached PR would be helpful not only to prevent the unnecessary redirects, but also to add support for <15.3 Safari versions. Thanks for the additional info!

coronasafe/care_fe#1060

@gigincg Error keys are now title cased. Since we already use `lodash` , I just used the built-in functions for this change.
We no longer need regex to replace the "_" with " ".

![image](https://user-images.githubusercontent.com/3626859/168886930-2575fa74-a956-46ec-bef0-d0882c38568f.png)

![image](https://user-images.githubusercontent.com/3626859/168886490-cddda095-ccbb-4480-a383-d3b1594f8e2a.png)


@arpancodes Hey, I tested it on Safari 15.3 and it was not working, on redirects, the Auth header is omitted which leads to some API calls failing.

Where did you test it? Are you sure you used a Safari browser? If so, which version?

![image](https://user-images.githubusercontent.com/3626859/168882769-6f1dd621-0944-4168-8340-b0daa005cc18.png)
![image](https://user-images.githubusercontent.com/3626859/168883039-307acfae-55bd-4846-96ee-17eac5ad14df.png)


> ![psajfojasf](https://user-images.githubusercontent.com/57055998/168846833-114760c8-196e-4a57-9a5d-f6584e94e526.jpg)
> 
> @Ashesh3 the alignment of created, diagnosis, and verified by is off, when we compared to the top

Fixed the alignment of the said elements.

![image](https://user-images.githubusercontent.com/3626859/168849734-e6e1fd04-93d0-42d3-8d82-06fb52802c87.png)


@nihal467 Fixed the left alignment, reduced the facility name font size, made it semi bold to match it more with the surrounding elements.

![image](https://user-images.githubusercontent.com/3626859/168834981-13034df9-ca39-49e0-878a-72318d84b0cb.png)


coronasafe/care_fe#2395

@nihal467 Fixed the alignment to match the padding of the patient name on the left side.

- With inner padding on div `md:pl-2` (Updated in this PR)
![image](https://user-images.githubusercontent.com/3626859/168526664-cfe8044f-de58-4f7b-a519-e74a9eb8bf95.png)

- Without inner padding (Not in this PR)
![image](https://user-images.githubusercontent.com/3626859/168526988-14165535-04bc-467b-96bd-4cf6426fa77c.png)

Let me know which one works the best.

> ![wpojefeopf](https://user-images.githubusercontent.com/57055998/168523635-7be54252-a901-40f1-89fb-a50e9d6a56c3.jpg)
> 
> @Ashesh3 @gigincg i can't see any update in the PR,

That's correct, Just wanted to get a visual feedback first if that looks good or not. if yes, I'd add the updated commit to this PR! Did that just now!

@nihal467 Is this better?

![image](https://user-images.githubusercontent.com/3626859/168482478-268b881d-e7f1-44a4-b9a1-0f0757bae21a.png)


Make Daily Round clone error more user friendly

coronasafe/care#734

Fix Profile Tab not showing active in sidebar

coronasafe/care_fe#2307

coronasafe/care_fe#2305

• PRs: <https://github.com/coronasafe/care_fe/pull/2442|#2442> <https://github.com/coronasafe/care_fe/pull/2433|#2433>
• Created Issues: <https://github.com/coronasafe/care_fe/issues/2440|2440> <https://github.com/coronasafe/care_fe/issues/2418|#2418>  

Submitted a PR, <https://github.com/coronasafe/care_fe/pull/2406>
Took up an issue to work on

Made a PR: <https://github.com/coronasafe/care_fe/issues/1060>

No update, just picked up an issue to consider working on.

Haven't had any new updates on my side. I'm just resting down and waiting for my pull requests to get merged.

Submitted 3 more PRs to the care_fe repo, resolving issues.
Submitted 1 PR to the care repo, requiring backend change.

Cloned and set up the `care_fe` repo locally successfully.
Submitted my first PR resolving an issue.

Issues raised: <https://github.com/coronasafe/care_fe/issues/2507|#2507> <https://github.com/coronasafe/care_fe/issues/2505|#2505>
PRs: <https://github.com/coronasafe/care/pull/789|care#789> <https://github.com/coronasafe/care/pull/788|care#788> <https://github.com/coronasafe/care_fe/pull/2536|#2536> <https://github.com/coronasafe/care_fe/pull/2525|#2525>

Worked on existing issues, now looking for new issues.

Started working on <https://github.com/coronasafe/care_fe/issues/2505|#2505>

Made two minor PRs, <https://github.com/coronasafe/care_fe/pull/2614|#2614> <https://github.com/coronasafe/care_fe/pull/2615|#2615>

Fix Bed actions button style

Fix error format for external test

Improve UI on External Test page

Disallow empty notes and comments

Fix API calls on Safari

Ashesh Kumar

GDC Intern

EOD Updates

Pull Requests Opened

Pull Requests Merged

Pull Requests Reviewed

Issues Opened

Comments Created

Markdown functionality for contributor bio

coronasafe/leaderboard#22

## Markdown Preview: 
![image](https://user-images.githubusercontent.com/40627011/170942870-596cf055-fa50-4f52-809c-d71c40195aab.png)


Fixed alignment of buttons: Patient Dashboard

coronasafe/care_fe#2555

coronasafe/care_fe#2519

coronasafe/care_fe#2520

@gigincg I've made the requested changes.

coronasafe/care_fe#2398

Fixed improper pagination padding in external results page

coronasafe/care_fe#2488

Pagination padding is improper in external results page

coronasafe/care_fe#2486

@nihal467 It is working for me...
Can you please share the link of the patricular page in which it is not working?

![image](https://user-images.githubusercontent.com/40627011/169691092-137d657b-951a-4028-99cf-7b4290e6a674.png)


coronasafe/care_fe#2474

Added appropriate message when no data is found

Display appropriate message when no data is found

coronasafe/care_fe#2403

Renamed button from 'Add COVID Suspect/Patient' to 'Add Patient'

Made the header of the resources page responsive

coronasafe/care_fe#2396

Should the page heading also be renamed to "Add Patient" ?

![image](https://user-images.githubusercontent.com/40627011/168822108-c4668a54-b05f-4756-a8a6-d39abfeff2cf.png)


coronasafe/care_fe#2393

Fixed loading error in investigation summary page

coronasafe/care_fe#2342

Renamed the page heading  from 'Covid Suspect Details' to 'Patient Details'

coronasafe/care_fe#2328

Submitted a PR <https://github.com/coronasafe/care_fe/pull/2474|#2474>

Working on #<https://github.com/coronasafe/care_fe/issues/2403|2403>

Submitted a PR <https://github.com/coronasafe/care_fe/pull/2398|#2398> and worked on issue <https://github.com/coronasafe/care_fe/issues/2403|#2403>

Made a PR <https://github.com/coronasafe/care_fe/pull/2396|#2396> and took up an issue to work on

Successfully set up care repo locally and looking for issues.

Caught up with college work.. couldn't do much, still working on <https://github.com/coronasafe/care_fe/issues/2517|#2517>

Working on <https://github.com/coronasafe/care_fe/issues/2517|#2517>

Submitted a PR <https://github.com/coronasafe/care_fe/pull/2555|#2555>

Fixed issues and resolved conflicts on previously made PRs.

Submitted a PR <https://github.com/coronasafe/leaderboard/pull/22|https://github.com/coronasafe/leaderboard/pull/22>

Working on <https://github.com/coronasafe/care_fe/issues/2588|#2588>


### Hi there 👋

Hi, I'm a 19 year old engineering undergrad pursuing B.Tech Information Technology at SSNCE.
I'm an aspiring web developer and software engineer 👨‍💻.

<br/>

---

<br/>

[Website 🌍](https://gokulramghv.github.io/)

<br/>

[Email 📬](mailto:gokulharividhya@gmail.com)


Gokulram A

worked on <https://github.com/coronasafe/care/issues/755|#755>

No updates on my side... Just relaxing for now.

EOD
Progressed to the last round on the hackathon :D

Krishna Sivakumar

coronasafe/care_fe#2619

The data's in the notification panel are conjusted.

coronasafe/care_fe#2618

coronasafe/care#807

Updated occupation field for text support

coronasafe/care#809

coronasafe/care#806

coronasafe/care#805

added backslashes at the end of each route in api.ts

coronasafe/care_fe#2581

coronasafe/care_fe#2580

coronasafe/leaderboard#20

coronasafe/care#797

coronasafe/care#791

coronasafe/care#793

coronasafe/care#743

coronasafe/care#751

coronasafe/care#770

API endpoint for https://github.com/coronasafe/care_fe/issues/2288

coronasafe/care#787

Switched Static Files in Care to WhiteNoise

coronasafe/care#724

updated `/add_user` endpoint to return status code instead of user object

coronasafe/care#718

add middleware_address field to facility table

coronasafe/care#784

Issue while running locally at migration 0293

coronasafe/care#780

closing PR for removing unnecessary commits

coronasafe/care#740

fix issues when qr_code_id is a blank string and show message for unique integrity error

coronasafe/care#777

added localstack s3 buckets for local environment

coronasafe/care#752

added a new created_by field in user model

coronasafe/care#746

coronasafe/care#763

coronasafe/care#756

![image](https://user-images.githubusercontent.com/44599895/168966739-c61aa973-9710-46f8-af98-e1895ff36e61.png)


coronasafe/care_fe#2291

coronasafe/care#748

coronasafe/care#747

Resource page header issue in handheld devices

coronasafe/care_fe#2370

Padding issue in external_results page mobile view

coronasafe/care_fe#2369

coronasafe/care_fe#2366

Added soft delete API endpoint PR: https://github.com/coronasafe/care/pull/740

coronasafe/care_fe#2288

Added soft delete API endpoint for file uploads

added patient details in investigation report

I can work for backend to create this API... which users can delete the file? @mathew-alex @gigincg 

coronasafe/care#735

I think everything works fine with backend and frontend, there might be an issue with celery, redis, or email configuration in deployment.  
- an example Log that celery gave as mail output while testing:

```
/bin/sh: 1: /usr/bin/google-chrome-stable: not found

[2022-05-11 18:51:39,662: WARNING/ForkPoolWorker-4] Content-Type: multipart/mixed; boundary="===============1608515035946104170=="

MIME-Version: 1.0

Subject: Patient Discharge Summary

From: Coronasafe network <care@coronasafe.network>

To: marmikpatel250@gmail.com

Date: Wed, 11 May 2022 13:21:39 -0000

Message-ID: <165227529966.20.13104182714356782510@63d8167dd5ba>


--===============1608515035946104170==

Content-Type: text/html; charset="utf-8"

MIME-Version: 1.0

Content-Transfer-Encoding: 7bit


Please find the attached file

--===============1608515035946104170==

Content-Type: application/pdf

MIME-Version: 1.0

Content-Transfer-Encoding: base64

Content-Disposition: attachment;

 filename="Srinivas Ramanujan-Discharge_Summary.pdf"



--===============1608515035946104170==--

[2022-05-11 18:51:39,663: WARNING/ForkPoolWorker-4] -------------------------------------------------------------------------------

[2022-05-11 18:51:39,664: INFO/ForkPoolWorker-4] Task care.facility.tasks.patient.discharge_report.generate_discharge_report[bd1ba65f-2148-4159-be73-b525466e40f9] succeeded in 5.204375585999969s: None
``` 




coronasafe/care_fe#2298

nothing from my side... day went like rocket

sorry, i can't attend EoD today, created a pr: <https://github.com/coronasafe/care/pull/770>

Filed two issues in care_fe about responsiveness, nothing else...

Nothing much from me today... Just preparing for practical exams

Working for Facility Cover Image Upload API endpoint

nothing much, just worked with my existing PR... working for DELETE method for file_upload

time past in exploring the backend only.... working on patient discharge summary email bug

Reviewed PRs, created draft PR for soft delete file API

EoD:
All time past in tour... No work today

Update:
Created PR: <https://github.com/coronasafe/care/pull/791>

EoD Update:
• Worked on existing PR: <https://github.com/coronasafe/care/pull/791>
• revised django testing and read about pytest

EoD update:
Filed issue and worked for the same: 
<https://github.com/coronasafe/care_fe/issues/2580|https://github.com/coronasafe/care_fe/issues/2580>
<https://github.com/coronasafe/care_fe/pull/2581|https://github.com/coronasafe/care_fe/pull/2581>

Working for occupation field in patient model

Made a PR: <https://github.com/coronasafe/care/pull/809|#809>

Added Facility Cover image API endpoints

added DELETE endpoint for file_upload

Updated Documentation

    

I am always passionate about learning and applying something new. My fields of interest are building and deploying Web Apps. Apart from coding, I also like to play chess a lot. 


Marmik Patel

feat: changed layout of individual user cards in user management page

coronasafe/care_fe#2590

> @Pragati1610 @developedBySJ @gigincg is it good for testing

Yes

coronasafe/care_fe#2455

Ease the tedious process of opening and viewing notifications sidebar in mobile view

coronasafe/care_fe#2463

Collapse notification side bar when visit link is clicked on any notification

coronasafe/care_fe#2462

fix: show the user that has created that particular user

fix: add loader to external results section

Top Contributors of the week