Skip to content

Change Open AI to Open AI (or API Compatible) #13608

Merged
subhramit merged 9 commits into
JabRef:mainfrom
SalvadorRomo:fix-for-issue-13585
Jul 29, 2025
Merged

Change Open AI to Open AI (or API Compatible) #13608
subhramit merged 9 commits into
JabRef:mainfrom
SalvadorRomo:fix-for-issue-13585

Conversation

@SalvadorRomo

Copy link
Copy Markdown

Closes #13585

This PR changes the the OpenAI label to OpenAI (Or Api Provider) inside the AI preference tab in the provider dropdown menu. Moreover, when generating the AI Summary, in the processing page, the message has ben changed to use the model selected.

Steps to test

1- Go to Preference
2- Select AI Tab
3- Select OpenAPI(or Api Compatible)
4- Mark the expert setting
5- Deploy a LLM locally through LM Studio o llama.cpp
6- Add the corresponding url in the field
7- Import a PDF and into the SummaryTab while is being processed you will see the summary is being genetated by the model selected.
8- Moreover, in the AI Chat at the bottom also you could notice that the chat is being powered by Opend API (or Api compatible)

image image image

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • Change in CHANGELOG.md described in a way that is understandable for the average user (if change is visible to the user)
  • [/] Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (if change is visible to the user)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • [/] Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

@SalvadorRomo SalvadorRomo changed the title Fix for issue 13585 Change Open API to Open API (or API Cpmpatible) Jul 29, 2025
@SalvadorRomo SalvadorRomo changed the title Change Open API to Open API (or API Cpmpatible) Change Open API to Open API (or API Compatible) Jul 29, 2025
Siedlerchr
Siedlerchr previously approved these changes Jul 29, 2025
@Siedlerchr Siedlerchr added this pull request to the merge queue Jul 29, 2025
@Siedlerchr Siedlerchr removed this pull request from the merge queue due to a manual request Jul 29, 2025
Comment thread CHANGELOG.md Outdated

### Added

- We change the Open API message to Open API (or API compatible). [#13585](https://github.com/JabRef/jabref/issues/13585)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

move to changed section

@Siedlerchr Siedlerchr added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Jul 29, 2025

@subhramit subhramit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please change the PR title and changelog entry as there seems to be confusion between OpenAI, API and Open API, Open API being something entirely different.

Comment thread CHANGELOG.md Outdated
Siedlerchr
Siedlerchr previously approved these changes Jul 29, 2025
Comment thread CHANGELOG.md Outdated
@Siedlerchr Siedlerchr changed the title Change Open API to Open API (or API Compatible) Change Open AI to Open AI (or API Compatible) Jul 29, 2025
@subhramit subhramit enabled auto-merge July 29, 2025 18:09
@trag-bot

trag-bot Bot commented Jul 29, 2025

Copy link
Copy Markdown

@trag-bot didn't find any issues in the code! ✅✨

@subhramit subhramit added this pull request to the merge queue Jul 29, 2025
Merged via the queue into JabRef:main with commit 7f75384 Jul 29, 2025
1 check passed
Siedlerchr added a commit that referenced this pull request Aug 2, 2025
* 'main' of github.com:JabRef/jabref: (26 commits)
  Change Open AI to Open AI  (or API Compatible)  (#13608)
  Remove all assignment tracking labels
  Updated architecture and components link in CONTRIBUTING.md (#13606)
  #13301 Fix/unicode reference marks (#13590)
  Show fetch exception at citation relation (#13549)
  More jbang caching
  Sequential run for moving issues
  Update CSL Styles (#13601)
  Fix gradle issues (#13603)
  Add System.exit (#13510)
  New Crowdin updates (#13600)
  Automatic lookup DOI at citation information. (#13596)
  Update heylogs from 0.10.0 to 0.11.1 (#13594)
  add fallback and reenable defaultcomeslast (#13593)
  Fix debian vagrant image (#13589)
  Fix: Replace string "Tests" with "Source Code Tests" in PR Comment (#13587)
  Fix version for SNAPSHOT (#13581)
  Update JDK and JavaFX (#13580)
  Add icon (#13579)
  Fixes link I broke in PR 13568 (#13577)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename "OpenAI" provider to "OpenAI (or API compatible)"

3 participants