Skip to content

Conversation

andreyfurdylo
Copy link
Contributor

@andreyfurdylo andreyfurdylo commented Jul 29, 2025

Replace this text with a description of your proposed change.

JOBS-8469

Add Glossary Entry suggestion status

Squash Commit Guidelines

This is a reminder on how to format your sqaush commit message. Delete from your PR after noting below.

Message Header

The message header is a single line that contains succinct description of the change containing a type: subject. For example:

feat: add Foo API

Allowed Types

This describes the kind of change that this commit is providing:

  • feat (when adding feature)
  • fix (when performing a bug fix)
  • docs (when adding or changing documentation)
  • refactor (when doing non-breaking code refactoring)
  • test (when adding missing tests)
  • chore (when performing maintainance)

Subject Text

This is a very short description of the change.

  • use imperative, present tense: “change” not “changed” nor “changes”
  • no dot (.) at the end

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new suggestionStatus field to glossary entries, allowing tracking of whether entries are "SUGGESTED" or "APPROVED". This enhancement supports the workflow for managing glossary entry suggestions in the Smartling platform.

  • Added suggestionStatus field to the GlossaryEntryBaseResponsePTO class
  • Updated test data to include the new field in JSON response
  • Added test assertion to verify the field is properly parsed

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
GlossaryEntryBaseResponsePTO.java Added suggestionStatus field with documentation
EntryManagementApiTest.java Updated test data and assertions to include suggestionStatus field

@andreyfurdylo andreyfurdylo merged commit b90102f into master Jul 31, 2025
2 checks passed
@andreyfurdylo andreyfurdylo deleted the glossary-entry-suggestion-status branch July 31, 2025 09:09
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.

4 participants