Changes from Transparent Path #356

glind · 2024-02-19T22:14:52Z

Purpose

Short description of the problem or feature

Approach

How does this change address the problem?

Further Info

Ticket number: #000

@mentions of the person or team responsible for reviewing proposed changes.

Add complexity estimate

This reverts commit d2f78ef.

Revert "Fix issue for retrieve query by uuid"

Updated Bravado Core version

…ganization Added default radius for organization

Added Gunicorn config

* squashed migration files * construct shipment_url only when shipment_id is present * remove shipment related from email if shipment is not available

…ation Handle when no custody organization mapped to custodian

* Change warning timezone to user's timezone only when core user has timezone * append ('UTC') for UTC timezone

* removed 'Captured at' from alert message * stop sending datetime in alert message

* github action for unit test * update triggers for unit test github action * github action for dev docker image build and push * github action for demo docker image build and push * github action for production docker image build and push * added pre-commit hooks for dev, demo and prod branch * update name for unit_test * Github Actions for Unit Test (#114) * Gunicorn timeout configuration * Fixes done for flake

Bumps the pip group group in /requirements with 6 updates: | Package | From | To | | --- | --- | --- | | [django](https://github.com/django/django) | `2.2.10` | `3.2.24` | | [django-filter](https://github.com/carltongibson/django-filter) | `2.2.0` | `2.4.0` | | [djangorestframework](https://github.com/encode/django-rest-framework) | `3.9.4` | `3.11.2` | | [requests](https://github.com/psf/requests) | `2.25.0` | `2.31.0` | | [aiohttp](https://github.com/aio-libs/aiohttp) | `3.5.4` | `3.9.2` | | [ipython](https://github.com/ipython/ipython) | `7.2.0` | `8.10.0` | Updates `django` from 2.2.10 to 3.2.24 - [Commits](django/django@2.2.10...3.2.24) Updates `django-filter` from 2.2.0 to 2.4.0 - [Release notes](https://github.com/carltongibson/django-filter/releases) - [Changelog](https://github.com/carltongibson/django-filter/blob/main/CHANGES.rst) - [Commits](carltongibson/django-filter@2.2.0...2.4.0) Updates `djangorestframework` from 3.9.4 to 3.11.2 - [Release notes](https://github.com/encode/django-rest-framework/releases) - [Commits](encode/django-rest-framework@3.9.4...3.11.2) Updates `requests` from 2.25.0 to 2.31.0 - [Release notes](https://github.com/psf/requests/releases) - [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md) - [Commits](psf/requests@v2.25.0...v2.31.0) Updates `aiohttp` from 3.5.4 to 3.9.2 - [Release notes](https://github.com/aio-libs/aiohttp/releases) - [Changelog](https://github.com/aio-libs/aiohttp/blob/master/CHANGES.rst) - [Commits](aio-libs/aiohttp@v3.5.4...v3.9.2) Updates `ipython` from 7.2.0 to 8.10.0 - [Release notes](https://github.com/ipython/ipython/releases) - [Commits](ipython/ipython@7.2.0...8.10.0) --- updated-dependencies: - dependency-name: django dependency-type: direct:production dependency-group: pip-security-group - dependency-name: django-filter dependency-type: direct:production dependency-group: pip-security-group - dependency-name: djangorestframework dependency-type: direct:production dependency-group: pip-security-group - dependency-name: requests dependency-type: direct:production dependency-group: pip-security-group - dependency-name: aiohttp dependency-type: direct:production dependency-group: pip-security-group - dependency-name: ipython dependency-type: direct:production dependency-group: pip-security-group ... Signed-off-by: dependabot[bot] <[email protected]>

The entire file was commented out, checking in without comments.

Fix 3.9,.2 aiohttp not found error from dependabot

…nts/pip-security-group-a44f67f13b Bump the pip group group in /requirements with 6 updates

glind · 2024-04-05T22:03:04Z

Have fun reviewing this...

glind · 2024-04-05T22:09:10Z

.github/workflows/demo-build.yml

+    name: Build and Push to GCR
+    runs-on: ubuntu-latest
+    env:
+      IMAGE_NAME: gcr.io/spry-bricolage-298920/transparent-path/demo/buildly-core


We need to make these paths consistent or remove these files from the upstream

glind

PIP style guide changes based on opinion are driving me crazy on this

glind · 2024-04-05T22:09:26Z

.github/workflows/dev-build.yml

+    name: Build and Push to GCR
+    runs-on: ubuntu-latest
+    env:
+      IMAGE_NAME: gcr.io/dev-buildly/transparent-path/buildly-core


same path problem

glind · 2024-04-05T22:09:43Z

.github/workflows/prod-build.yml

+    name: Build and Push image to GCR
+    runs-on: ubuntu-latest
+    env:
+      IMAGE_NAME: gcr.io/spry-bricolage-298920/transparent-path/prod/buildly-core


path problem

glind · 2024-04-05T22:12:06Z

buildly/settings/authentication.py

@@ -81,11 +78,13 @@
 SOCIAL_AUTH_URL_NAMESPACE = 'social'
 SOCIAL_AUTH_POSTGRES_JSONFIELD = True

-SOCIAL_AUTH_REDIRECT_IS_HTTPS = True if os.getenv('SOCIAL_AUTH_REDIRECT_IS_HTTPS') == 'True' else False
+SOCIAL_AUTH_REDIRECT_IS_HTTPS = (


so many of these line break changes I want to fire myself and make whoever did these work on all the conflicts. Let's write a single style guide for pythonic changes and only make them during refactor releases.

Greg Lind added 30 commits May 8, 2020 15:43

Update ISSUE_TEMPLATE.md

8446525

Add complexity estimate

update travis configs

8b1bb38

update travis configs

6fddbe6

Merge remote-tracking branch 'upstream/master'

3bd0c35

move build to travis script

109c646

update travis for aws

4a75e0d

update travis for aws

c4ea278

update travis for aws

5f82439

update travis for aws

6bb2130

update travis for aws

813e3b8

update travis for aws

5c4f196

update travis for aws

284c823

update travis for aws

565b580

update travis for aws

d5e7808

update travis for aws

3e719c8

cleanup initital data

731212d

cleanup initital data

1ad1966

cleanup initital data

b148c9e

remove organization_name

a80a7d5

remove organization_name

b31fc81

update travis var

d35d082

update travis var

c9c2d96

update travis var

aec2f69

revert keys

a05d287

fix org

71fbd8c

fix org

9d572a3

fix org

e90cf33

fix org

c42f1d4

fix org

326eff3

fix org

9aee00a

vishalajackus and others added 25 commits August 12, 2021 19:26

Fix issue for retrieve query by uuid (#88)

d2f78ef

Revert "Fix issue for retrieve query by uuid (#88)"

823eedd

This reverts commit d2f78ef.

Merge pull request #90 from TransparentPath/revert-88-datamesh-uuid-fix

0aa608a

Revert "Fix issue for retrieve query by uuid"

Handled boolean for CORS_ORIGIN_ALLOW_ALL

269d76b

Updated Bravado Core version

e471e84

Merge pull request #95 from TransparentPath/fix/bravado-core-spec

006f70b

Updated Bravado Core version

Return response data only for PUT, POST, DELETE (#97)

9df8e88

Added default radius for organization

dfb361e

Merge pull request #100 from TransparentPath/fix#99/default-radius-or…

3f1b727

…ganization Added default radius for organization

Allow unlimited line size for request

3b25820

Merge pull request #102 from TransparentPath/fix/gunicorn-config

711fbe9

Added Gunicorn config

sensor service email alert for unassigned moving sensor (#105)

87c45d2

* squashed migration files * construct shipment_url only when shipment_id is present * remove shipment related from email if shipment is not available

Handle when no custody organization mapped to custodian

f6ab185

Merge pull request #107 from TransparentPath/fix/custodian-no-organiz…

eceb280

…ation Handle when no custody organization mapped to custodian

Environmental warning timezone. (#108)

2492d66

* Change warning timezone to user's timezone only when core user has timezone * append ('UTC') for UTC timezone

Remove Travis

c79fba8

Remove timestamp from alert messages (#110)

b444bd1

* removed 'Captured at' from alert message * stop sending datetime in alert message

Gunicorn timeout configuration

5b3fc68

Remove dev deployment setup

0961ca6

Setup github actions

352d71f

Update dev-build.yml

4f0337d

The entire file was commented out, checking in without comments.

Update base.txt

61e2651

Fix 3.9,.2 aiohttp not found error from dependabot

Merge pull request #131 from TransparentPath/dependabot/pip/requireme…

95b09ff

…nts/pip-security-group-a44f67f13b Bump the pip group group in /requirements with 6 updates

glind requested a review from patelradhika April 5, 2024 22:02

glind commented Apr 5, 2024

View reviewed changes

patelradhika force-pushed the master branch from 97621c3 to 95b09ff Compare May 20, 2024 08:22

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Changes from Transparent Path #356

Changes from Transparent Path #356

glind commented Feb 19, 2024

glind commented Apr 5, 2024

glind Apr 5, 2024

glind left a comment

glind Apr 5, 2024

glind Apr 5, 2024

glind Apr 5, 2024

Changes from Transparent Path #356

Are you sure you want to change the base?

Changes from Transparent Path #356

Conversation

glind commented Feb 19, 2024

Purpose

Approach

Further Info

glind commented Apr 5, 2024

glind Apr 5, 2024

Choose a reason for hiding this comment

glind left a comment

Choose a reason for hiding this comment

glind Apr 5, 2024

Choose a reason for hiding this comment

glind Apr 5, 2024

Choose a reason for hiding this comment

glind Apr 5, 2024

Choose a reason for hiding this comment