From 78ae0fe57bd307c453ad338eb820791b8123f690 Mon Sep 17 00:00:00 2001 From: "github-actions[bot]" Date: Sun, 19 Nov 2023 01:59:07 +0000 Subject: [PATCH] =?UTF-8?q?=F0=9F=8C=8D=20Update=20JSON=20-=2019=20Nov=202?= =?UTF-8?q?023=20|=2007:29=20AM?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- data/github/ayush-seth.json | 18 ++++++++++++++++-- data/github/nihal467.json | 9 ++++++++- data/github/rithviknishad.json | 24 +++++++++++++++++++----- data/github/shyamprakash123.json | 17 ++++++++++++----- data/slack/message_dump.json | 3 ++- 5 files changed, 57 insertions(+), 14 deletions(-) diff --git a/data/github/ayush-seth.json b/data/github/ayush-seth.json index cb421128..743ece5f 100644 --- a/data/github/ayush-seth.json +++ b/data/github/ayush-seth.json @@ -1,6 +1,20 @@ { - "last_updated": 1700239006.0, + "last_updated": 1700301288.0, "activity": [ + { + "type": "comment_created", + "title": "coronasafe/care_fe#6665", + "time": 1700301288.0, + "link": "https://github.com/coronasafe/care_fe/issues/6665#issuecomment-1817463174", + "text": "This is a bigger problem at its core with how route protection and login is being handled and just replacing `window.location.href =` with `navigate()` is not going to solve these issues. \r\n\r\nFor ex. when I am logged out and I navigate to `/shifting/board`, instead of being redirected to `/login` I stay on `/shifting/board` and in those cases, after logging in, it does `window.location.href = window.location.pathname.toString()`, using `navigate` here won't do anything because the path does not change, you have to reload the whole page. Same goes for any other url." + }, + { + "type": "comment_created", + "title": "coronasafe/care_fe#6665", + "time": 1700300406.0, + "link": "https://github.com/coronasafe/care_fe/issues/6665#issuecomment-1817458427", + "text": "Any reason this project is using [raviger](https://github.com/kyeotic/raviger) which barely anyone has ever heard of and has not been updated since 2022 instead of using the industry standard [React Router](https://github.com/remix-run/react-router)?" + }, { "type": "comment_created", "title": "coronasafe/care_fe#6503", @@ -41,7 +55,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6663", "title": "show delete user button only for stateAdmin", - "stale_for": 0, + "stale_for": 1, "labels": [ "needs testing" ] diff --git a/data/github/nihal467.json b/data/github/nihal467.json index 741a019f..5ea71cda 100644 --- a/data/github/nihal467.json +++ b/data/github/nihal467.json @@ -1,6 +1,13 @@ { - "last_updated": 1700135059.0, + "last_updated": 1700318591.0, "activity": [ + { + "type": "comment_created", + "title": "coronasafe/care_fe#6650", + "time": 1700318591.0, + "link": "https://github.com/coronasafe/care_fe/issues/6650#issuecomment-1817526924", + "text": "@konavivekramakrishna use the login :\r\nusername ;nihal-district / Password : Test@123\r\n" + }, { "type": "comment_created", "title": "coronasafe/care_fe#6655", diff --git a/data/github/rithviknishad.json b/data/github/rithviknishad.json index 94dca00a..5ed7f986 100644 --- a/data/github/rithviknishad.json +++ b/data/github/rithviknishad.json @@ -1,6 +1,20 @@ { - "last_updated": 1700239764.0, + "last_updated": 1700330752.0, "activity": [ + { + "type": "comment_created", + "title": "coronasafe/care_fe#6650", + "time": 1700330752.0, + "link": "https://github.com/coronasafe/care_fe/issues/6650#issuecomment-1817585406", + "text": "@konavivekramakrishna The API has responded with 202 Accepted. Hence no errors were encountered. Have you tried calling the list API for the same after a successful upsert?" + }, + { + "type": "comment_created", + "title": "coronasafe/care_fe#6665", + "time": 1700303828.0, + "link": "https://github.com/coronasafe/care_fe/issues/6665#issuecomment-1817472008", + "text": "I don't see any issue with Raviger's routing.\r\n\r\nIf you look at the code, we have two routers: Session and App router. App router is accessible only if the query to get the current profile responds with 200, you'll be exposed to the App router, else it'll show the Session router.\r\n\r\nWe just need to invoke the refetch of the current user query somehow whenever the JWT access token is updated in the local storage. That'd cause the AuthUserProvider component to render, and hence exposing the appropriate router based on the response.\r\n\r\n\"image\"\r\n\r\n\"image\"\r\n\r\n" + }, { "type": "issue_opened", "title": "coronasafe/care_fe#6665", @@ -27151,7 +27165,7 @@ { "link": "https://github.com/coronasafe/care/pull/1704", "title": "Adds authz for Prescription & Medicine Administrations", - "stale_for": 3, + "stale_for": 4, "labels": [ "waiting-for-review" ] @@ -27159,7 +27173,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6464", "title": "Adds user type: Nurse and Nurse (readonly)", - "stale_for": 3, + "stale_for": 4, "labels": [ "changes required", "waiting for back end" @@ -27168,7 +27182,7 @@ { "link": "https://github.com/coronasafe/care/pull/1678", "title": "Patient Consultation: Route to Facility \ud83c\udfe5, Admission Date & Time \ud83d\udd5a, Rename variable: `verified_by` to `treating_physician`", - "stale_for": 3, + "stale_for": 4, "labels": [ "waiting-for-review" ] @@ -27176,7 +27190,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6458", "title": "Patient Consultation: Route to Facility \ud83c\udfe5, Admission Date & Time \ud83d\udd5a", - "stale_for": 3, + "stale_for": 4, "labels": [ "Backend", "tested", diff --git a/data/github/shyamprakash123.json b/data/github/shyamprakash123.json index 2d1333b0..719523ba 100644 --- a/data/github/shyamprakash123.json +++ b/data/github/shyamprakash123.json @@ -1,6 +1,13 @@ { - "last_updated": 1700234724.0, + "last_updated": 1700270740.0, "activity": [ + { + "type": "comment_created", + "title": "coronasafe/care_fe#6608", + "time": 1700270740.0, + "link": "https://github.com/coronasafe/care_fe/issues/6608#issuecomment-1817309126", + "text": "> Which user can delete is already present in the code base. The issue is about hiding the button instead of keeping it disabled.\r\n> \r\n> \"image\"\r\n\r\nUnderstood, please assign me so that I can start working on this." + }, { "type": "comment_created", "title": "coronasafe/care_fe#6608", @@ -444,7 +451,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6662", "title": "Removed mandatory for Rhythm field", - "stale_for": 0, + "stale_for": 1, "labels": [ "needs testing" ] @@ -452,7 +459,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6578", "title": "\ud83e\ude9b Fixed Non-uniform height of Asset List Tiles.", - "stale_for": 0, + "stale_for": 1, "labels": [ "tested", "good to merge" @@ -461,7 +468,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6576", "title": "\u2714\ufe0f Fixed reloading issue while Opening Investigations or Treatment Summary in Consultation Dashboard.", - "stale_for": 0, + "stale_for": 1, "labels": [ "needs testing" ] @@ -469,7 +476,7 @@ { "link": "https://github.com/coronasafe/care_fe/pull/6574", "title": "\ud83d\ude80 Improved info of vitals monitor in mobile view.", - "stale_for": 0, + "stale_for": 1, "labels": [ "tested", "good to merge" diff --git a/data/slack/message_dump.json b/data/slack/message_dump.json index 634b1d31..a0bd804d 100644 --- a/data/slack/message_dump.json +++ b/data/slack/message_dump.json @@ -93632,5 +93632,6 @@ ] } ], - "2023-11-18 00:00:00": [] + "2023-11-18 00:00:00": [], + "2023-11-19 00:00:00": [] } \ No newline at end of file