coronasafe/care#835

coronasafe/care#833

Update deploy workflow to actually run tests

coronasafe/care#832

coronasafe/care#831

Edit/ create Discharge summary even after discharge

coronasafe/care_fe#564

coronasafe/care_fe#370

coronasafe/care_fe#2386

coronasafe/care_fe#2563

> 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 

coronasafe/care_fe#2639

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

coronasafe/care_fe#2638

> 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

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

Added the 404 page PR.
Nothing else for the day.

Made 2 PRs, solved 2 issues.
Currently travelling so can't join EOD call :sweat_smile:

Submitted 2 PRs that solve 3 issues. I have defined Code Climate and Deep Scan as my enemies.

Set up care_fe locally, solving some line ending warnings

Updates :
made issues - <https://github.com/coronasafe/care_fe/issues/2562> and <https://github.com/coronasafe/care_fe/issues/2561>
made pr - <https://github.com/coronasafe/care_fe/pull/2563>

Updates : made PR <https://github.com/coronasafe/care_fe/pull/2599|#2599> and recommitted <https://github.com/coronasafe/care_fe/pull/2563|#2563>. Working on issue <https://github.com/coronasafe/care_fe/issues/2562|#2562>

EOD update :
working on <https://github.com/coronasafe/care_fe/issues/2603|#2603> and <https://github.com/coronasafe/care_fe/issues/2602|#2602>

EOD UPDATE -

Made PRs - <https://github.com/coronasafe/care_fe/pull/2639|2639> and <https://github.com/coronasafe/care_fe/pull/2638|2638>
Mostly explored Care Backend and working out how we can divide testing (edited)

Nothing much for the day, just went through old issues

EOD Update :

Worked with creating testing workflow and setting up testing branch for backend

Issues :
<https://github.com/coronasafe/care/issues/832>
<https://github.com/coronasafe/care/issues/833>

PR :
<https://github.com/coronasafe/care/pull/835>
<https://github.com/coronasafe/care/pull/831> (Epic PR)

Added tests workflow

Testing

fixed word wrapping in patient page tabs


# 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

> 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

coronasafe/care#826

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

coronasafe/care#836

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`

coronasafe/care#772

Its possible to do it on the BE but the min_quantity will still be redundant atlest in `FacilityInventoryLogSerializer`

> @sainAk the min_quantity can stay and maybe FE can ignore the flag if the min_quantity value is not present, or we can add that logic into our flagging system.

Sure I'll apply the changes

For now I'm only building the docker image, after we add localstack we can switch over to compose for testing.

coronasafe/care#768

Modified maximum number of doses for a patient

coronasafe/care#775

This issue is caused because `Asset QR Code ID` is sending blank string which causes integrity error
@rabilrbl do you want to take up this issue to handle it on the FE?

coronasafe/care_fe#2422

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

coronasafe/care#773

@gigincg this is not reproduceable locally

coronasafe/care_fe#2452

fix: Inventory unexpectedly marked as Low Stock

add migration to change meta.camera_address to meta.local_ip_address

coronasafe/care#771

coronasafe/care_fe#2437

This issue can be combined with coronasafe/care_fe#2285

coronasafe/care_fe#2435

Merge base dockerfile into main dockerfile and add layer caching in github actions

coronasafe/care#754

coronasafe/care#753

added localstack s3 buckets for local environment

coronasafe/care#752

@Marmik2003 you changes from other PR are being mixed up in ths one
To fix the problem:
Close this PR 
create a new branch from fd027983dcdf6ce436b5499183a57ac24dbedd98 and cherry-pick the changes on that branch.

coronasafe/care#735

@cp-Coder tailwind SS/custom-forms needs to be migrated to tailwindcss/forms https://tailwindcss.com/blog/tailwindcss-v2#utility-friendly-form-styles

Top Contributors of the week