-
Notifications
You must be signed in to change notification settings - Fork 334
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/GitHub 677/identity migration #728
Conversation
1058e81
to
887b92a
Compare
from .types import DynamoProjectMetadata | ||
|
||
|
||
class DynamoProjectMetadataWrapper: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like the logic in this wrapper can absorbed into the DynamoProjectMetadata
dataclass itself? We can use a classmethod to get a given metadata record from dynamo and load it into the dataclass.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes remaining.
api/environments/dynamodb/tests/types/test_dynamodb_project_metadata.py
Outdated
Show resolved
Hide resolved
api/environments/dynamodb/tests/types/test_dynamodb_project_metadata.py
Outdated
Show resolved
Hide resolved
Add command to migrate identity to edge Add migrate identity to sales dashboard tests(command): add test for migrate_to_edge command * tests(sales-dashboard): Add tests for migrate identity
Add support for migrating the identity data to dynamo db using the following two ways:
python manage.py migrate_to_edge <project_id>
UI:
Also implements: #726
Deployment Notes: Create a dynamodb table with id(Number) as partition key and use it's name for
PROJECT_METADATA_TABLE_NAME_DYNAMO
Env vairable.