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

NextJS Fullstack Webapp #149

Merged
merged 4 commits into from
Sep 16, 2023
Merged

NextJS Fullstack Webapp #149

merged 4 commits into from
Sep 16, 2023

Conversation

srbhr
Copy link
Owner

@srbhr srbhr commented Sep 16, 2023

Pull Request Title

Adds in a Fullstack web app for resume matcher.
Stack:

  • NextJS
  • FastAPI

Related Issue

Description

This feature adds in an

Type

  • Bug Fix
  • Feature Enhancement
  • Documentation Update
  • Code Refactoring
  • Other (please specify):

Proposed Changes

  • Full Stack Webapp

Screenshots / Code Snippets (if applicable)

None

Checklist

  • The code compiles successfully without any errors or warnings
  • The changes have been tested and verified
  • The documentation has been updated (if applicable)
  • The changes follow the project's coding guidelines and best practices
  • The commit messages are descriptive and follow the project's guidelines
  • All tests (if applicable) pass successfully
  • This pull request has been linked to the related issue (if applicable)

Additional Information

Sayvai and others added 4 commits August 28, 2023 21:29
Adds in a nextjs and fastapi webapp (experimental)
* feat: render save service keys on web app ui

Implemented new feature to allow users to view the required configurable service keys (and urls) to set, on the web app UI.

The configurable required service keys data is dynamically derived from the backend master YAML config file (located in `/scripts/similarity/config.yml`), and its actual pre-saved secret key values derived from a localised programatically created git-ignored YAML secrets config file (located in `/scripts/similarity/config.local.yml`).

Whenever a key is no longer required by the app, then it is safe to remove key from the master config file, and in turn this key and any pre-saved secret value will no longer render on the web UI.

If a secret has yet to be set for any of the required keys, then the value will fallback to the default placeholder value defined in the master YAML config.

* chore: update next.js version and dependencies

* chore: error handling for missing keys config file

Add error handling on the backend api for when `similarity/config.yaml` does not exist, to which it responds with a 404 not found response.

As a result, also added error handling on the frontend ui code to handle the above 404 backend api response edge case to not render the Service Keys ui section.

* chore: ui refactor resume.tsx to hero.tsx

Logically abstracted out and renamed `Hero` component to its own folder, and corrected / renamed component filename.

* chore: sync requirements.txt from upstream

Pulled a bugfix for `requirements.txt` file from upstream fork origin repo main branch, which broke the webapp backend server run.

Root cause was that the qdrant-client dependency was set with a specific version; `qdrant-client>=1.1.1`, but the upstream reversion fix; `qdrant-client` without explicitly specifying a version resolved the webapp backend server run issue.
This change is to add new sections labelled `Debugging` and `Troubleshooting` to the `webapp/README.md` file.

The debugging section guides users on how to run the FastAPI backend server in VS Codes' debug mode, along with running the frontend server in isolation.

The troubleshooting section currently highlights one potential common issue that users may encounter when only running the frontend server in isolation (e.g. `npm run next-dev`).
@srbhr srbhr merged commit 2b9cb6a into main Sep 16, 2023
2 checks passed
@srbhr srbhr deleted the resume-matcher-fullstack-webapp branch September 25, 2023 10:59
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