Hide Configure Asset button if `!asset_class`

coronasafe/care_fe#2654

Adds Asset Class field and Improve Asset Type option's descriptions

coronasafe/care_fe#2653

coronasafe/care_fe#2427

@sainAk @iamsdas @vigneshhari Still receiving it as `meta.camera_address` :(

![image](https://user-images.githubusercontent.com/25143503/172047923-15a86fa7-e5d0-4f02-8f2a-5b7688dac1ef.png)


coronasafe/care#759

coronasafe/care_fe#2552


https://user-images.githubusercontent.com/25143503/172046804-90c6228a-8f37-4f16-b859-972c61e68407.mp4



@nihal467 unable to reproduce. this is what i see, even from preview deploy. will look into it.

![image](https://user-images.githubusercontent.com/25143503/172045750-1654c71d-96fe-4be1-93cc-2826956df84b.png)


Hide the Configure button if asset_class = null

coronasafe/care_fe#2649

@iamsdas working now! Great :)
@sainAk will you be moving this to a new issue as the original issue is now solved.

coronasafe/care#792

Getting a response of 500 when attempting to create asset with asset_class specified.

Weren't the migrations run automatically? It's a char field in the last change. 

@iamsdas I'm using the staging API via proxy mentioned in `care_fe/package.json`

I'm yet to setup a local care BE on my machine.

![image](https://user-images.githubusercontent.com/25143503/171849660-749941b9-314e-494f-b7df-696037c63ac8.png)

- What I'm performing is:
```sh
curl 'https://careapi.coronasafe.in/api/v1/asset/' \
        -X POST \
        -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:100.0) Gecko/20100101 Firefox/100.0' \
        -H 'Accept: application/json, text/plain, */*' \
        -H 'Accept-Language: en-US,en;q=0.5' \
        -H 'Accept-Encoding: gzip, deflate, br' \
        -H 'Content-Type: application/json' \
        -H 'Authorization: Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ0b2tlbl90eXBlIjoiYWNjZXNzIiwiZXhwIjoxNjU0MjU4MjY5LCJqdGkiOiIwZjgyMTQ5Mzc4MjY0ZWUyYTJmNzVlNDFmYzllMmRjMSIsInVzZXJfaWQiOjIwMn0.gp1grpzzKDH3ZCPoRSI_XTLX-IoZnWDATUcZQpolwuU' \
        -H 'Origin: http://localhost:4000' \
        -H 'Connection: keep-alive' \
        -H 'Referer: http://localhost:4000/facility/4fa7d5bf-1862-4ebb-b2d1-8284d12185ac/assets/new' \
        -H 'Sec-Fetch-Dest: empty' \
        -H 'Sec-Fetch-Mode: cors' \
        -H 'Sec-Fetch-Site: same-origin' \
        --data-raw ' 
{                              
  "name": "Hello Asset :wave:",
  "asset_type": "INTERNAL",
  "asset_class": "ONVIF",
  "description": "",   
  "is_working": "true",    
  "not_working_reason": "",
  "serial_number": "",   
  "warranty_details": "",                            
  "location": "afa28ab1-53f5-4554-8cb0-77c1e5d58c37",
  "vendor_name": "", 
  "support_name": "", 
  "support_email": "",             
  "support_phone": "+917777777777",
  "qr_code_id": "",
  "cancelToken": {
    "promise": {}
  }
}'

```

- And I'm getting the following for the above.
![image](https://user-images.githubusercontent.com/25143503/171851175-e385b0d2-8d30-419c-889e-e7e3b61b5e1f.png)

- However, if I just remove `asset_class` attribute in the same above cURL command:
```sh
curl 'https://careapi.coronasafe.in/api/v1/asset/' \
        -X POST \
        -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:100.0) Gecko/20100101 Firefox/100.0' \
        -H 'Accept: application/json, text/plain, */*' \
        -H 'Accept-Language: en-US,en;q=0.5' \
        -H 'Accept-Encoding: gzip, deflate, br' \
        -H 'Content-Type: application/json' \
        -H 'Authorization: Bearer eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ0b2tlbl90eXBlIjoiYWNjZXNzIiwiZXhwIjoxNjU0MjU4MjY5LCJqdGkiOiIwZjgyMTQ5Mzc4MjY0ZWUyYTJmNzVlNDFmYzllMmRjMSIsInVzZXJfaWQiOjIwMn0.gp1grpzzKDH3ZCPoRSI_XTLX-IoZnWDATUcZQpolwuU' \
        -H 'Origin: http://localhost:4000' \
        -H 'Connection: keep-alive' \
        -H 'Referer: http://localhost:4000/facility/4fa7d5bf-1862-4ebb-b2d1-8284d12185ac/assets/new' \
        -H 'Sec-Fetch-Dest: empty' \
        -H 'Sec-Fetch-Mode: cors' \
        -H 'Sec-Fetch-Site: same-origin' \
        --data-raw '
{
  "name": "Hello Asset :wave:",
  "asset_type": "INTERNAL",
  "description": "",
  "is_working": "true",
  "not_working_reason": "",
  "serial_number": "",
  "warranty_details": "",
  "location": "afa28ab1-53f5-4554-8cb0-77c1e5d58c37",
  "vendor_name": "",
  "support_name": "",
  "support_email": "",
  "support_phone": "+917777777777",
  "qr_code_id": "",
  "cancelToken": {
    "promise": {}
  }
}'
```

- I'm getting a good response for that:
![image](https://user-images.githubusercontent.com/25143503/171851568-8201dd1e-38a0-4a1e-bd3a-367eb724f981.png)


Hide "copy logs from previous logs" if first consultation update

coronasafe/care_fe#2644

@vigneshhari @iamsdas still getting 500 as response :/
Am I doing something wrong or...?

![image](https://user-images.githubusercontent.com/25143503/171834317-b5c65b96-dec5-469f-997f-4c221bb5f892.png)


Adds hints for delete last entry, mark as accident and unmark actions in inventory management

coronasafe/care_fe#2485

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

coronasafe/care_fe#2581

Closing as re-worked with new changes on #52. Ocean of merge conflicts in this.

coronasafe/leaderboard#35

coronasafe/leaderboard#52

replace "Doctor Video" to "Doctor Connect"

coronasafe/care_fe#2518

coronasafe/leaderboard#48

Fixes unintended border in focused inputs

coronasafe/care_fe#2622

coronasafe/care_fe#2560

@anuran-roy I've cleared cache and triggered deploy. but still it's failing :/ 
I can also see that you do have the latest develop here...

![image](https://user-images.githubusercontent.com/25143503/171428721-c7f6d99f-43d8-44da-8189-1da93380ecdf.png)


coronasafe/care_fe#2543

@developedBySJ it becomes visible in staging deploy once empty cache and hard reload is done.

Or can github workflow itself do it alone? 🤔 

coronasafe/leaderboard#32

Can't nextjs itself handle it?
How about this?
We'll make an API endpoint in nextjs that'll dispatch an email with sendgrid or anything.
And invoke that endpoint every week by a github workflow? 😎 

How about sending a weekly report to mail about their activity last week to mail?

Support for multiple contributors to be listed in top contributors

coronasafe/leaderboard#42

Top contributors card automatically takes first element even when no contribution.

coronasafe/leaderboard#40

@anuran-roy deploy preview is failing. merge the latest develop branch.

@skks1212 deploy preview is failing. merge the latest develop branch.

coronasafe/care_fe#2563

@Pranshu1902 deploy preview is failing. merge the latest develop branch.

coronasafe/care_fe#2499

@Marmik2003 deploy preview is failing. merge the latest develop branch.

@rabilrbl deploy preview is failing. merge the latest develop branch.

coronasafe/care_fe#2567

coronasafe/care_fe#2570

@kunatastic deploy preview is failing. merge the latest develop branch.

coronasafe/care_fe#2579

coronasafe/care_fe#2586

@developedBySJ 
Tried to trace where the issue is coming from, but no clue...
Was able to reproduce this issue when reviewing `cra-5` branch #2527 

**Steps to reproduce:**

```sh
git checkout develop
git fetch upstream
git pull upstream develop
yarn start
```
On firefox 100.0.2
![image](https://user-images.githubusercontent.com/25143503/171105319-368a8c7d-67be-454e-b659-7eb96a1f5408.png)


@developedBySJ 
The text fields seems to have additional unintended border when focused.
Also it strikes through the labels.

Home page's leaderboard title is statically typed

coronasafe/leaderboard#36


https://user-images.githubusercontent.com/25143503/170995258-2de66c3f-caba-4119-9585-3b9a425a25a0.mp4



coronasafe/care_fe#2470

Fix activity card being hidden in `md` sizes

coronasafe/leaderboard#19

@gigincg updated padding

![image](https://user-images.githubusercontent.com/25143503/170855096-81b78e28-ff17-42aa-83ad-53c13c457740.png)


coronasafe/leaderboard#17

![image](https://user-images.githubusercontent.com/25143503/170854596-b792ff64-421b-4eaa-83dd-45c622a3f435.png)


Handle `issue_closed` GitHub activity in renderText

coronasafe/leaderboard#15

Activity card goes behind Contributor Info Card in some sizes

coronasafe/leaderboard#18

coronasafe/leaderboard#16

Activity type "issue_closed" is not handled in renderText

coronasafe/leaderboard#14

coronasafe/leaderboard#12

Hides "currently working on" if data is empty

coronasafe/leaderboard#11

Responsive alignment for InfoCard Content

coronasafe/leaderboard#9

Contributor profile page width not correct in mobile.

coronasafe/leaderboard#10

InfoCard content incorrectly aligned in different window size

coronasafe/leaderboard#8

coronasafe/care_fe#2564

coronasafe/care_fe#2565

Netlify preview does show up with current date time for me.

![image](https://user-images.githubusercontent.com/25143503/170827410-dc225c9e-b9c1-4893-8337-322ed5e314d5.png)


coronasafe/care_fe#2473

Creates a new Field in CreateAsset Form to update the asset_class for the asset

@gigincg should this field be a required field?

coronasafe/care_fe#2431

coronasafe/care_fe#2510

coronasafe/care_fe#2511

coronasafe/care_fe#2466

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

coronasafe/care_fe#2465

Improves UI in Facility Create Page for Oxygen Related Fields

coronasafe/care_fe#2464

Feature: Keyboard Shortcuts for Camera Live Feed

coronasafe/care_fe#2381

@gigincg I'm thinking of taking this up.

Two approaches can be there.

### 1. Client Side
Where each preset is traversed and selected at an interval controlled by the client side.
However in this method the middleware will not be aware whether a client is using patrol mode or not, and hence other clients who might access the same camera will also not be aware of this. Set preset requests may conflict the order if more than one client accesses it.

### 2. Server side
An endpoint for the camera can be exposed to enable patrol mode, and also status response can include whether the resource is in patrol mode or not. Hence every client will be aware whether it's in patrol mode or not. Presets are traversed and selected by the middleware/backend, hence order is maintained regardless of multiple clients accessing the resource. Preset traverse interval can be optionally adjusted by client.

coronasafe/care_fe#2160

Inventory Delete Last Entry and Mark as Accident's intended use case can be confusing

coronasafe/care_fe#2483

@rabilrbl got it, my bad, didn't see that.
@gigincg this PR is also resolving closing the drawer also when notification's visit is clicked. I have merged rabil's #2423 into this.

Sourcing .env causes command not found errors

coronasafe/care_fe#2469

Incosistent results in sidebar for languages other than english

coronasafe/care_fe#2438

- Added shortcut keys for patient feed.
- Added help menu that lists all shortcut keys (in both asset cam feed and patient feed)

![image](https://user-images.githubusercontent.com/25143503/169040603-acae41b9-3f3d-45ba-8c7b-2f4e3ae93104.png)

![image](https://user-images.githubusercontent.com/25143503/169040697-0e763ba1-59ca-4b71-a76e-e00325138141.png)


- [ ] `Ctrl` + `Up`: Move Camera Up
- [ ] `Ctrl` + `Left`: Move Camera Left
- [ ] `Ctrl` + `Down`: Move Camera Down
- [ ] `Ctrl` + `Right`: Move Camera Right
- [ ] `Alt` + `S`: Save / Update Preset
- [ ] `Alt` + `X`: Stop feed
- [ ] `Alt` + `R`: Reset Feed
- [ ] `Alt` + `+`: Zoom In
- [ ] `Alt` + `-`: Zoom Out
- [ ] `F`: Enter full-screen (Exit full-screen shortcut is handled by browser itself)

Any suggestions? @gigincg 

coronasafe/care_fe#2374

@cp-Coder [here](http://localhost:4000/facility/05980e8f-8860-4460-b863-fefdff1d187b/patient/e45b93eb-4b1a-4f91-9b6f-3533b719e9a6/consultation/008e06f1-20fa-44a1-b665-bb1d9b21d04e/daily_rounds/72e37048-f117-4c23-aed2-6c77558b09f1/update) inside ABG Analaysis.

coronasafe/care_fe#2257

Reload Patient Presets in PatientFeed View When updating preset

coronasafe/care_fe#2336

Changes "Patient Details" to "Patient Dashboard" in Consultation Page

coronasafe/care_fe#2327

Monthly leaderboard

Fixes missing quotes in .env and update REACT_APP_TITLE



**⚡ An Electrical Engineer by degree and a Software Developer by passion.**

*Passionate about creating scalable and distributed systems for the power grid and interested in contributing to open source digital public goods.*

Rithvik Nishad

GDC Intern

EOD Updates

![image](https://user-images.githubusercontent.com/59426397/172102055-3b2f1de7-34ca-46b9-a5e3-f907c3e22201.png)

@nihal is it okay now?

coronasafe/care_fe#2641

Disabled dates before previous bed's start date while switching patients bed

coronasafe/care_fe#2647

> ![1](https://user-images.githubusercontent.com/57055998/171985994-d143e8e9-e4eb-42e9-b0d3-7acf35641587.jpg) @patelaryan7751 @gigincg
> 
> the red highlight colour is partially colored, can you fix that

yeah sure @nihal467 

Overiding the rule "Comment should contain atleast 1 character" and having empty comments in Shifting Deatils page

coronasafe/care_fe#2532

Hey, @khavinshankar I have removed the unused commented lines instead of the console.log().

coronasafe/care_fe#2419

Fixed the unhandled situation of QR code Scanner

Fixed the unresponsive behaviour of UI in sample test page

coronasafe/care_fe#2490

Hey, @khavinshankar I have removed all commented unused variables.

Fixed the Ui of Patient page unresponsive behaviour in mobile

coronasafe/care_fe#2493

Fixed the Ui unresposive behaviour of Shifting page in mobile devices

coronasafe/care_fe#2528

Fixed the button getting hidden in mobile screen of Patient Feed Page and Asset Configuration Page

coronasafe/care_fe#2558

made the splitting lines visible in the patient page

@khavinshankar i have resolved the merge conflicts

Hey, @nihal467 the deploy preview is ready now.

coronasafe/care_fe#2613

@khavinshankar resolved the merge conflict

Prevented sending empty notification to facilities

coronasafe/care_fe#2542

Fixed the unresponsive behaviour of Resource(List View) page

coronasafe/care_fe#2531

Splitting lines are missing in the patient tab.

coronasafe/care_fe#2630

I think the patient details page is already fixed @nihal467 :thinking: 

![image](https://user-images.githubusercontent.com/59426397/171566780-6bb0946f-a216-42fc-bbf4-1bbcee09465d.png)


coronasafe/care_fe#2616

Prevented the shifting page and resource page details to look congested

coronasafe/care_fe#2629

Fixed the hover effect issue in sample details button

coronasafe/care_fe#2620

@gigincg I think if we give "hover:bg-gray-100" then the effect would not be visible as it was in the previous behavior. 

The shifting page and resource page details are so conjested, it is having no gap in between the lines

coronasafe/care_fe#2627

Removed white spaces of the filters from all eligible pages

coronasafe/care_fe#2623

Added spacing between elements to prevent User Profile Details from looking congested

coronasafe/care_fe#2625

User profile details page are so conjusted without space

coronasafe/care_fe#2624

All the page filters are utilizing a lot of white space on the right side

coronasafe/care_fe#2621

Hover Effect of Sample Details Button not visible in Sample Mnagement System Page

coronasafe/care_fe#2617

@rithviknishad 

I am also having this issue in the Chrome browser

![image](https://user-images.githubusercontent.com/59426397/171413635-d66b5e22-88c5-4d7b-b720-daab8ed2af18.png)




Fixed BP Decimal Precision to 2 decimal places

https://github.com/coronasafe/care_fe/pull/2525 this issue has been fixed by @Ashesh3 

coronasafe/care_fe#2559

Hey @Pranshu1902 i have mentioned this bug in this issue https://github.com/coronasafe/care_fe/issues/2557 

Required Fields can be submitted empty in some forms

coronasafe/care_fe#2557

Bug in Consultation Form Page required fields can be updated being empty.

coronasafe/care_fe#2553

Hey @Pranshu1902 I have already made a Pull Request on this issue https://github.com/coronasafe/care_fe/pull/2493

coronasafe/care_fe#2547

Prevent empty notifications while notifying a facility.

coronasafe/care_fe#2541

coronasafe/care_fe#2525

> 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 :smile: 

Hey @Ashesh3  i found a bug in the shifting page comment posting algorithm we can easily overide the check **"Comment should contain 1 character"** by giving **space** . You can view it here #2532 for more detail demonstration. 

I think we need to also have a check for spaces it might go like this:

```
if (noteField.trim().length < 1) {
      Notification.Error({
        msg: "Note Should Contain At Least 1 Character",
      });
      return;
    }
```

Some of the buttons get hidden in Live feed page in some mobile devices.

coronasafe/care_fe#2530

@kunatastic hey are you working on this issue #2529  regarding the resource page  ? Can i work on that part ?

Top Contributors of the week