Skip to content

Implement home page rendering and update requirements - #28

Open
RishobGuhaPatra wants to merge 1 commit into
CloudBytes-Academy:p2-byog-flaskfrom
RishobGuhaPatra:p2-byog-flask
Open

Implement home page rendering and update requirements#28
RishobGuhaPatra wants to merge 1 commit into
CloudBytes-Academy:p2-byog-flaskfrom
RishobGuhaPatra:p2-byog-flask

Conversation

@RishobGuhaPatra

Copy link
Copy Markdown

No description provided.

Copilot AI lite review requested due to automatic review settings August 13, 2026 16:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Flask application to render the existing home-page template and implements a basic /search route, alongside simplifying/updating the pinned Python dependencies.

Changes:

  • Render templates/index.html for the / route using render_template.
  • Implement /search to redirect users to a Google search based on the q query parameter.
  • Reduce and update pinned dependencies in requirements.txt.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 3 comments.

File Description
requirements.txt Simplifies and updates pinned runtime dependencies.
app.py Implements template rendering for / and adds Google redirect behavior for /search.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread app.py
Comment on lines +7 to +9
# """
# TODO: Render the home page provided under templates/index.html in the repository
# """
Comment thread app.py
Comment on lines +14 to +18
# """
# TODO:
# 1. Capture the word that is being searched
# 2. Seach for the word on Google and display results
# """
Comment thread app.py
Comment on lines +20 to +21
args = request.args.get("q")
return redirect(f"https://www.google.com/search?q={args}")
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.

2 participants