coronasafe/care_fe#2691

coronasafe/care_fe#2690

@nihal467 I have changed the camera feed UI completely, and also added space for the 5 para monitor readings, which will be added in the future. I have removed the aspect ratio class so that the flex could work properly, but it will be of no problem as the video will just show black bars if out of ratio.

coronasafe/care_fe#2687

Added an alert when no CSV file is uploaded in Upload External Results Page.

coronasafe/care_fe#2689

Fixed Malfunction : Pressure Sore history

coronasafe/care_fe#2688

@nihal467 should work now, merged develop

coronasafe/care_fe#2638

Center aligned the pagination on Results page Issue#2680

coronasafe/care_fe#2684

Changed allowed hosts to wildcard in staging

coronasafe/care#852

coronasafe/care#848

Align bed assignment modal and improve general UI

coronasafe/care_fe#2682

Hey @patelaryan7751 can you add this as a commit to #2563 and close this pr ?
Because the UI will be completely changing with that PR so...

coronasafe/care_fe#2675

coronasafe/care_fe#2677

> @skks1212 @gigincg is it good for testing !

Yep! 👍

coronasafe/care_fe#2672

coronasafe/care#851

@khavinshankar hopefully that solves it. 

coronasafe/care_fe#2563

Solve hidden overflow in duplicate patient record modal

coronasafe/care_fe#2665

> @skks1212 please do work on the merge conflict

@khavinshankar done

coronasafe/care_fe#2639

Added spaces on resource page in mobile view

coronasafe/care_fe#2669

coronasafe/care#841

Yep, we have to make sure that all user information is intact.
The text on the forgot password page has been changed in PR [care_fe #2665](https://github.com/coronasafe/care_fe/pull/2665)

coronasafe/care#842

coronasafe/care#843

Change password reset page text to indicate email will be send 

coronasafe/care_fe#2664

Update deploy workflow to actually run tests

coronasafe/care#832

coronasafe/care#840

coronasafe/care#835

coronasafe/care#833

coronasafe/care#831

Edit/ create Discharge summary even after discharge

coronasafe/care_fe#564

coronasafe/care_fe#370

coronasafe/care_fe#2386

> patient detail page alignment is off

will be fixed in #2563 

coronasafe/care_fe#2616

all the critical summary heading are misaligned like summary, update etc.. buttons 
fixed in #2639 

Removed App.css and index.scss, moved css imports to index.css

> please do work on the requested changes

@khavinshankar Done please check

replaced all occurrences of whitespace-no-wrap to whitespace-nowrap (tailwind migration)

coronasafe/care_fe#2599

@kunatastic UI is completely changing in [https://github.com/coronasafe/care_fe/pull/2563](https://github.com/coronasafe/care_fe/pull/2563 )

coronasafe/care_fe#2634

coronasafe/care_fe#2540

@nihal467 I feel we should not center align it as per design. If you say we should, il add a commit, but I personally think this looks better and more "profiley"

> please do work on the requested changes

I don't have access to the cdn, and i think we don't require a cdn for this PR @gigincg what should I do?

@gigincg deploy is failing, can we clear cache?

sounds good, can you make it run from a dummy email for now? Il help

coronasafe/leaderboard#32

> How about sending a weekly report to mail about their activity last week to mail?

It's a nice idea, but we are not going to get a backend so... : (

coronasafe/leaderboard#34

coronasafe/leaderboard#33

coronasafe/leaderboard#12

coronasafe/leaderboard#13

![image](https://user-images.githubusercontent.com/23238460/170844160-6bee6941-0379-43ed-b995-e4e976d78fd2.png)
Able to recreate on desktop. Error occurs due to no specification of break word for wrapping text.

coronasafe/leaderboard#10

Converted external links to anchor tags and updated Shivank Kacker's details

coronasafe/leaderboard#5

coronasafe/care_fe#2322

Updated UI for Patient info card. Also shifted patient discharge buttons under the info card

coronasafe/care_fe#2562

coronasafe/care_fe#2561

coronasafe/care_fe#2521

@cp-Coder please try to resolve the issues asap! A lot of UI overhauls need the latest tailwind dist. 😅

coronasafe/care_fe#2402

> Instead of the asset folder let's pull it from a cdn

Cool! Can you put it up on the coronasafe cdn? (If we have one?)
Otherwise I will put up on another.

coronasafe/care_fe#2405

> @skks1212 Do make a separate issue for handling the buttons properly if you haven't done that yet

Yes doing it!


coronasafe/care_fe#2351

> @skks1212 This behaviour preserves the page that the user is at when they log back in. If we solve this, we might have to preserve the previous page (Maybe in localstorage/sessionstorage) in order to redirect them back to the page.

I think the best preservation technique would be to have a ?ref query param in the url containing the url of the page to redirect to after logging in. I have seen many websites do this.

coronasafe/care_fe#2376

> @skks1212 can you make this into a single page html with css and we can put this globaly on our domain. this looks good
Suree! There you go @tomahawk-pilot [https://github.com/skks1212/care_misc/tree/master/error_pages]( https://github.com/skks1212/care_misc/tree/master/error_pages)

URL does not change when redirected to login

coronasafe/care_fe#2375

> @skks1212 Can you make it "CARE" instead of "care"
> 
> Also, can you make an issue to better handle the size of the buttons on the right when all buttons aren't being shown

Sure! Do I make a new PR for the buttons or do I amalgamate that in this one?

coronasafe/care_fe#2352

Logging in does not indicate a loading symbol

coronasafe/care_fe#2349

> Issue already added and made PR #2338

Great, closing

Removed action buttons and indicated for discharged patients

coronasafe/care_fe#2350

Logging in does not have a loading symbol

remove log updates and edit consultation button in discharged patients card.

coronasafe/care_fe#2330

Log update with empty systolic value throws a render error.

coronasafe/care_fe#2329

Code Climate tells me to refactor the codebase for frequently used code blocks, but I think it will be unnecessary. Do let me know if I have to mandatorily make the changes. Otherwise I think it is fit to merge
 

Solved in [https://github.com/coronasafe/care_fe/pull/2322](https://github.com/coronasafe/care_fe/pull/2322)

coronasafe/care_fe#2316

coronasafe/care_fe#2317

> @skks1212 can you try making the time and rhythm column a bit narrower because they don't need that much space.

The other columns will automatically give space to the description columns if needed. I have not limited space for sake of responsiveness. 

coronasafe/care_fe#2313

> @skks1212 Add a Screenshot to the new UI!

Edited!
![image](https://user-images.githubusercontent.com/23238460/168007779-def91f2c-d51d-4596-b0f4-ba8c8d7f4c0d.png)


Issue resolved in [https://github.com/coronasafe/care_fe/pull/2315](https://github.com/coronasafe/care_fe/pull/2315)
Whitespace can be removed but from design and responsiveness perspective I think it is better as is.

coronasafe/care_fe#2046

Reduced the Edit consultation form length. Utilized grid.

coronasafe/care_fe#2315

coronasafe/care_fe#2258

Solved in https://github.com/coronasafe/care_fe/pull/2313

Updated Rhythm View to table for easy access and efficiency

Fix camera buttons

Testing


# Hey👋!!
I like making stuff that people love!
Currently I am interning for Coronasafe and Pupilfirst, freelancing, and working on [Writeroo](https://writeroo.in), a platform for readers and writers to come together and share the love!

✉️ shivank@writeroo.in

See you around!


Shivank Kacker

GDC Intern

EOD Updates

Pull Requests Opened

create a field validator for validating non empty fields

coronasafe/care#849

> Another thing worth mentioning is that we are being spammed by requests with unknown hosts which triggers `django.security.DisallowedHost` events on sentry these events are not worth tracking and they only consume events bandwidth Its possible to ignore these events from the deployment itself.

@vigneshhari whats your take on this?

coronasafe/care#844

Update sentry sdk and specify deployment environment for each deployment

@JAIMIN-CHOKHAWALA https://github.com/coronasafe/care_fe/pull/2665 will update the status message

showing password-reset email status success even when I don't have CARE user account. 

closing this issue as it's being tracked in https://github.com/coronasafe/care_fe/issues/2664 

coronasafe/care#836

coronasafe/care#838

fix docs pipeline and remove new pipeline

coronasafe/care#839

coronasafe/care#837

@skks1212 @Marmik2003 we can integrate the test step in the deployment workflow 
https://github.com/coronasafe/care/blob/master/.github/workflows/deployment.yaml#L54-L56

coronasafe/care#826

> Looks good, i think it is ok if we don't remove sphinx from local requirements as some developers might also want to change the docs after few major changes!

I've refrenced the docs.txt requirements in local.txt so sphinx will also get installed in local environments

@vigneshhari add https://pre-commit.ci/ to the repo

coronasafe/care#834

coronasafe/care#667

closed by 9e03d628f6bc657e02b2d266fea8f5dbf215a3a8

coronasafe/care#674

coronasafe/care#488

DRY Permissions added for assetLocation viewset

coronasafe/care#830

coronasafe/care#828

updated user tests, removed facility tests

coronasafe/care#824

@Marmik2003 there are a lot of changes that I want to make, I would suggest pushing the changes to a new branch in care itself `marmik2003/feat/uset-tests` and create a draft PR from that branch

Create pipelines to automatically build documentation

coronasafe/care#825

This change would be a 2 step process
1. Migrating the location field to normal fields 
2. removing references of the location filed from older migrations (0015, 0017)

coronasafe/care#822

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

coronasafe/care#718

> > > We can but let it be there. Since we have multiple instances being deployed to
> > 
> > 
> > @tomahawk-pilot I'm asking auto deployments only for Staging-egov, If the review requirement could be removed for `Staging-egov` environment the FE development and testing won't be blocked by pending deployments.
> 
> Feel free to remove the review requirements for staging egov environment

I don't have the privileges for it.

coronasafe/care#792

ValidationError raised by django give 500 response

coronasafe/care#819

Endpoint analyse/ fixed for read-only users

coronasafe/care#817

> We can but let it be there. Since we have multiple instances being deployed to

@tomahawk-pilot I'm asking auto deployments only for Staging-egov, If the review requirement could be removed for `Staging-egov` environment the FE development and testing won't be blocked by pending deployments.

It's waiting for deployment https://github.com/coronasafe/care/deployments/activity_log?environment=Staging-egov

@mathew-alex @tomahawk-pilot can you update the staging to deploy automatically?

The asset_class expects an integer
Exception: https://sentry.io/organizations/coronasafe-network/issues/3306110898/

> Which site

Not sure but it was cause by this worker `care-staging-celery-worker-7dfcc76ffd-gc8f8`

coronasafe/care#816

tracked in https://github.com/coronasafe/care/issues/816

coronasafe/care_fe#2298

coronasafe/leaderboard#47

coronasafe/care#812

Finalize MiddlewareAuthentication for required endpoints

coronasafe/care#811

coronasafe/care#791

Remove existing bots from github scrapper

coronasafe/leaderboard#45

coronasafe/care#807

coronasafe/care_fe#2570

@Pragati1610 delete the package-lock from your branch as we have started using yarn.

coronasafe/care_fe#2590

coronasafe/care_fe#2578

Update deploy workflow and fix issues in dockerfile

coronasafe/care_fe#2573

Update deploy workflow ad fix issues in dockerfile

Make the meta field optional in Asset & Bed Tables

coronasafe/care#769

coronasafe/care#795

There are relevent changes only the force push was caused by rebasing the branch to update the migrations

coronasafe/care#746

added a new created_by field in user model

Switched Static Files in Care to WhiteNoise

coronasafe/care#724

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

coronasafe/care#777

@siddnikh we can just fix the `UserSerializer` update method to handle password field changes, also make the password a writeonly field, by doing this we can just PATCH the password, I don't think there is a requirement for a separate endpoint.

coronasafe/care#779

This should be good to merge as now we are using a single docker image.

coronasafe/care#770

@cp-Coder update https://www.npmjs.com/package/rescript-material-ui as well

coronasafe/care_fe#2427

Added soft delete API endpoint for file uploads

coronasafe/care#740

what we can do is rename the min_quantity field to default_min_quantity in the response and decide the is_low based on the `FacilityInventoryMinQuantity`

Top Contributors of the week