Skip to content
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

Workshop Solutions - No merge needed #1

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

Workshop Solutions - No merge needed #1

wants to merge 9 commits into from

Conversation

tgrall
Copy link
Contributor

@tgrall tgrall commented Jun 15, 2024

This pull request includes significant changes to the Django-based application, primarily aimed at enhancing its functionality and usability. The changes include setting up a development container, updating the README, introducing new API routes and their corresponding tests, creating a new HTML home page, and adding sample data for Azure VMs.

API Endpoint Enhancements:

  • copilot/api/views.py: Added two new API endpoints: get_hello and get_vms. The get_hello endpoint returns a greeting message based on a query parameter, while the get_vms endpoint returns a list of VMs read from a local JSON file.
  • copilot/api/urls.py: Updated the URL configuration to include the new get_hello and get_vms endpoints.

Testing:

  • copilot/api/tests.py: Introduced several tests for the new get_hello and get_vms endpoints. The tests validate the endpoints' responses and their content.

Project Documentation:

  • README.md: Updated the project's README file to provide an overview of the project and instructions on how to get started. It also includes a brief description of the labs available in the project.
  • docs/001-implement-new-route.md, docs/002-data-and-services.md, docs/003-create-homepage.md: Added detailed lab instructions in these new markdown files. The labs guide users through the process of adding new API routes, generating sample data, testing APIs, and creating a homepage. [1] [2] [3]

Development Environment:

  • .devcontainer/devcontainer.json: Integrated GitHub Copilot into the development environment by adding it as an extension in the VS Code devcontainer configuration.

Homepage Creation:

Other Changes:

…with the message "Hello World" when the query parameter `key` is present. If the `key` query parameter is not present, return a JSON response with the message "key query parameter is required" and a status code of 501.

This commit adds the necessary code changes to the `api/views.py` file to implement the new `get_hello` function. It also includes corresponding tests in the `api/tests.py` file to ensure the endpoint behaves as expected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant