Skip to content

fix: remove duplicate LawGPT URL settings - #1619

Open
saurabhhhcodes wants to merge 1 commit into
viru0909-dev:mainfrom
saurabhhhcodes:fix/1600-remove-duplicate-lawgpt-url
Open

fix: remove duplicate LawGPT URL settings#1619
saurabhhhcodes wants to merge 1 commit into
viru0909-dev:mainfrom
saurabhhhcodes:fix/1600-remove-duplicate-lawgpt-url

Conversation

@saurabhhhcodes

@saurabhhhcodes saurabhhhcodes commented Jul 17, 2026

Copy link
Copy Markdown

Description

Removes the two duplicate lawgpt.service.url entries from application.properties.example, preserving the single environment-backed default. This prevents a later hard-coded value from overriding LAWGPT_SERVICE_URL.

Closes #1600

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature: not applicable to this configuration-only fix
  • Breaking change: not applicable; the environment-backed setting is preserved
  • Documentation update: not required for this configuration-only fix

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • Documentation changes are not applicable to this configuration-only fix
  • My changes generate no new warnings
  • Tests are not applicable to this configuration-only fix; focused validation is included below
  • New and existing validation passes locally

Validation

  • git diff --check passed.
  • Confirmed exactly one lawgpt.service.url entry remains and retains ${LAWGPT_SERVICE_URL:http://localhost:8001}.
  • Maven validate passed with tests skipped because this is configuration-only.

@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the CodeBlooded's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown
Contributor

Hi @saurabhhhcodes, thanks for contributing to Nyay Setu! 🎉

I have automatically:

  • 👤 Assigned this PR to you.
  • 🏷️ Applied the gssoc:approved label.

Our workflows will now analyze your changes to classify:

  • 📈 PR Difficulty: level:*
  • 🧩 PR Type: type:*
  • 🌟 PR Quality: quality:*

Tip

Ensure your PR description references the issue it resolves (e.g. Closes #123). This allows the bot to inherit any additional labels from that issue!

Happy coding! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Duplicate lawgpt.service.url property overrides environment-based configuration

1 participant