Skip to content

Prompt#24

Merged
ndayiemile merged 3 commits intomainfrom
prompt
Oct 5, 2025
Merged

Prompt#24
ndayiemile merged 3 commits intomainfrom
prompt

Conversation

@slageyuri
Copy link
Collaborator

@slageyuri slageyuri commented Oct 5, 2025

Index.htlm and index.css changes: adapated to all browsers, logout and reset password button. tests_history deleted.

Description by Korbit AI

What change is being made?

Apply small formatting cleanup and update the UI template:

  • Normalize exception handling formatting in gemini_model.py.
  • Update the HTML template to swap the order of action buttons, adjust CSS classes, and refresh footer text/year:
    • Change Password link moved to a new password-reset-btn class and placed after Logout.
    • Logout link moved to a dedicated logout-btn class and placed first.
    • Footer year updated from 2021 to 2025.

Why are these changes being made?

Refactor for consistent formatting and clearer UI affordances; align button semantics and styling with updated classes, and refresh the footer year.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues... but I did find this rooster.

   \\
   (o>
\\_//)
 \_/_)
  _|_
Files scanned
File Path Reviewed
resumax_backend/resumax_algo/templates/resumax_algo/index.html
resumax_backend/resumax_algo/gemini_model.py

Explore our documentation to understand the languages and file types we support and the files we ignore.

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@coveralls
Copy link

Pull Request Test Coverage Report for Build 18263115941

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 85.0%

Totals Coverage Status
Change from base Build 18261013410: 0.0%
Covered Lines: 646
Relevant Lines: 760

💛 - Coveralls

@ndayiemile ndayiemile merged commit 351991e into main Oct 5, 2025
2 checks passed
@ndayiemile ndayiemile deleted the prompt branch October 5, 2025 19:13
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.

3 participants