USACE / USACE/cwms-data-api

Move to latest Javalin

Open
#1,004 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

priority:medium
Dominant language
Java
Stars
28
Forks
25
Avg merge
4d 22h
Merged PRs (30d)
20

Description

Need to coordinate with district first and discuss about timelines but javalin 4.x is not maintain and 6.x contains some features we'd like, like case insensitive path matching. Would also drastically simplify our writing of queries by being able to use the var data type.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating the Javalin 4.x dependency and the current routing and query code. Review Javalin 6.x migration requirements and confirm the desired case-insensitive path matching and var-related query simplification with the district before changing scope. Done means the service runs on the latest agreed Javalin version and the requested features work without regressions.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api, backend
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.