From 0d0792e6d985af619ef7a390b334853ca48c92b7 Mon Sep 17 00:00:00 2001 From: Steve Carroll Date: Fri, 8 Mar 2024 09:06:35 -0500 Subject: [PATCH 1/2] Add dependabot PR docs --- SUMMARY.md | 1 + deployment/merging-dependabot-prs.md | 37 ++++++++++++++++++++++++++++ 2 files changed, 38 insertions(+) create mode 100644 deployment/merging-dependabot-prs.md diff --git a/SUMMARY.md b/SUMMARY.md index 1baf7448..f83158cb 100644 --- a/SUMMARY.md +++ b/SUMMARY.md @@ -16,6 +16,7 @@ * [Deploying PDP within the Code for Philly cluster](deployment/deploying-pdp-within-the-code-for-philly-cluster.md) * [Kubernetes Setup](setup/kubernetes-setup.md) * [Kubernetes logs](deployment/kubernetes-logs.md) + * [Merging Dependabot PRs](deployment/merging-dependabot-prs.md) * [Troubleshooting](troubleshooting/README.md) * [Common Errors](troubleshooting/common-errors.md) * [Dups Problem](troubleshooting/dups-problem.md) diff --git a/deployment/merging-dependabot-prs.md b/deployment/merging-dependabot-prs.md new file mode 100644 index 00000000..1d221565 --- /dev/null +++ b/deployment/merging-dependabot-prs.md @@ -0,0 +1,37 @@ +# Dependabot PRs +- [Dependabot PRs](#dependabot-prs) + - [Frontend Dependabot PRs](#frontend-dependabot-prs) + - [Backend Dependabot PRs](#backend-dependabot-prs) + +## Frontend Dependabot PRs +As the client facing part of the app is pretty minimal, this process should cover most frontend dependabot PRs. + +- Pull dependabot PR +``` +gh pr checkout [prNumber] +``` + +- Rebuild and run the container +``` +docker-compose down -v +docker-compose build +docker-compose up +``` + +- Log into `base_admin` user +- Go to `Admin` page + - Upload 2 Volgistics data CSVs in the same upload action + - Click `Run Data Analysis` +- Go to `Users` page + - Create new user + - Update user via `Update User` button + - Change user password via `Change Password` button +- Go to 360 Dataview + - Search for a common name + - Click the user to make sure page renders correctly +- Log out + +If the package patch notes look non-breaking and you encounter no errors in this process, the Dependabot PR should be safe to merge. + +## Backend Dependabot PRs +Caution should be exercised when updating backend libraries. \ No newline at end of file From 397a401e35cbf8586b00b1b0ba47a888fa17591f Mon Sep 17 00:00:00 2001 From: Steve Carroll Date: Fri, 8 Mar 2024 09:09:29 -0500 Subject: [PATCH 2/2] Small revision to dependabot pr docs --- deployment/merging-dependabot-prs.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deployment/merging-dependabot-prs.md b/deployment/merging-dependabot-prs.md index 1d221565..93038b5a 100644 --- a/deployment/merging-dependabot-prs.md +++ b/deployment/merging-dependabot-prs.md @@ -34,4 +34,4 @@ docker-compose up If the package patch notes look non-breaking and you encounter no errors in this process, the Dependabot PR should be safe to merge. ## Backend Dependabot PRs -Caution should be exercised when updating backend libraries. \ No newline at end of file +Documentation needed.