Skip to content

Conversation

@kmcginnes
Copy link
Collaborator

@kmcginnes kmcginnes commented Jan 22, 2026

Description

This PR updates user-facing terminology throughout the application to use "Query Language" instead of "Graph Type" for better clarity and accuracy.

Changes

UI Updates:

  • Connection creation form: "Graph Type" → "Query Language"
  • Connection detail view: "Graph Type" → "Query Language"
  • Connection row display: Updated to use new translation key
  • Import graph error messages: "graph type" → "query language"

Translation System:

  • Renamed translation key from graph-type to query-language across all three translation files (Gremlin, openCypher, SPARQL)
  • Updated all references to use the new query-language key

Documentation Updates:

  • Updated README.md environment variable description
  • Updated development.md section header to "Supported Graph Data Models and Query Languages"
  • Updated features documentation for connection setup
  • Updated getting started guide example

Test Updates:

  • Updated test expectations in ImportGraphButton.test.tsx to match new terminology
  • Enhanced useEdgeAttributesAsScalars.test.ts with separate test suites for Gremlin and openCypher to verify correct terminology ("Edge Label" vs "Relationship Type")

Rationale

"Query Language" more accurately describes what users are selecting (Gremlin, openCypher, or SPARQL) rather than the ambiguous "Graph Type" which could be confused with the data model (property graph vs RDF).

Validation

CleanShot 2026-01-22 at 16 14 02@2x CleanShot 2026-01-22 at 16 14 19@2x

Related Issues

Check List

  • I confirm that my contribution is made under the terms of the Apache 2.0
    license.
  • I have run pnpm checks to ensure code compiles and meets standards.
  • I have run pnpm test to check if all tests are passing.
  • I have covered new added functionality with unit tests if necessary.
  • I have added an entry in the Changelog.md.

@codecov
Copy link

codecov bot commented Jan 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.83%. Comparing base (c5affc5) to head (d7c490b).
⚠️ Report is 44 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1474       +/-   ##
===========================================
+ Coverage   47.81%   62.83%   +15.02%     
===========================================
  Files         382      315       -67     
  Lines        8525     6530     -1995     
  Branches     3159     2271      -888     
===========================================
+ Hits         4076     4103       +27     
+ Misses       3070     1696     -1374     
+ Partials     1379      731      -648     
Flag Coverage Δ
unittests 62.83% <100.00%> (+15.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@kmcginnes kmcginnes force-pushed the feature/labels-query-language branch from 1e6a043 to 57bee3f Compare January 22, 2026 22:09
@kmcginnes kmcginnes changed the title Use “query language” instead of “graph type” Update terminology: "Graph Type" → "Query Language" Jan 22, 2026
@kmcginnes kmcginnes force-pushed the feature/labels-query-language branch from 57bee3f to e16270a Compare January 22, 2026 22:25
@kmcginnes kmcginnes force-pushed the feature/labels-query-language branch from e16270a to d7c490b Compare January 22, 2026 22:33
@kmcginnes kmcginnes marked this pull request as ready for review January 22, 2026 23:12
@kmcginnes kmcginnes merged commit d30ad33 into aws:main Jan 22, 2026
4 checks passed
@kmcginnes kmcginnes deleted the feature/labels-query-language branch January 22, 2026 23:33
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.

Fix wording for graph type and query engine/language

2 participants