-
Notifications
You must be signed in to change notification settings - Fork 22
Add a microservices based sample app to test enablement #18
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
Conversation
Can you be more specific:
|
Starting a Django project using intellij generates the files. Intellij runs All the files were generated except docker specific files and The files that were modified are under the MainService folder for each microservice. The specific files are urls.py, views.py and the models.py. |
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.
Still need to review the urls.py
and views.py
files.
sample-applications/vehicle-dealership-sample-app/VehicleInventoryApp/my.cnf
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/ImageServiceApp.iml
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/ImageServiceApp/settings.py
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/db.sqlite3
Outdated
Show resolved
Hide resolved
...ons/vehicle-dealership-sample-app/VehicleInventoryApp/MainService/migrations/0001_initial.py
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/MainService/views.py
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/docker-compose.yaml
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/VehicleInventoryApp/MainService/models.py
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/VehicleInventoryApp/requirements.txt
Outdated
Show resolved
Hide resolved
...e-applications/vehicle-dealership-sample-app/VehicleInventoryApp/VehicleInventoryApp/urls.py
Outdated
Show resolved
Hide resolved
...pplications/vehicle-dealership-sample-app/ImageServiceApp/MainService/migrations/__init__.py
Outdated
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/ImageServiceApp/MainService/views.py
Show resolved
Hide resolved
sample-applications/vehicle-dealership-sample-app/VehicleInventoryApp/MainService/views.py
Outdated
Show resolved
Hide resolved
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.
For all code files, I think we need to add a header:
# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved. |
Since you are adding new folders, you should setup eachdist.py and eachdist.ini, to allow lint and test workflow apply to all files/tests.
Adding a microservices based sample app to help us test enablement for the python integration. Most of the code is boiler plate code created by Django.
The most important code to look at is inside the MainService folder for each microservice. Those are under the urls.py, views.py and the models.py.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.