Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Last non-technical editor of the scenario in the API response #7581

Open
wants to merge 18 commits into
base: staging
Choose a base branch
from

Conversation

mgoworko
Copy link
Contributor

@mgoworko mgoworko commented Feb 20, 2025

Describe your changes

Return (optionally) additional information from the processes API - about the last non-technical edit of the scenario.

There are multiple cases, where Nu scenarios are updated automatically - that includes migrations and automatic updates. As a result, the last user who edited the scenario is very often "Nussknacker".

This change introduces additional, optional, fields in the API response, with information about last non-technical user who edited the scenario. Only backend is modified, the additional information is not used on the FE yet.

Checklist before merge

  • Related issue ID is placed at the beginning of PR title in [brackets] (can be GH issue or Nu Jira issue)
  • Code is cleaned from temporary changes and commented out lines
  • Parts of the code that are not easy to understand are documented in the code
  • Changes are covered by automated tests
  • Showcase in dev-application.conf added to demonstrate the feature
  • Documentation added or updated
  • Added entry in Changelog.md describing the change from the perspective of a public distribution user
  • Added MigrationGuide.md entry in the appropriate subcategory if introducing a breaking change
  • Verify that PR will be squashed during merge

@github-actions github-actions bot added the docs label Feb 21, 2025
@mgoworko mgoworko marked this pull request as ready for review February 21, 2025 14:59
@mgoworko mgoworko force-pushed the scenario-business-editor branch from 4b60b23 to ba9b851 Compare February 25, 2025 11:29
…editor

# Conflicts:
#	designer/restmodel/src/main/scala/pl/touk/nussknacker/restmodel/scenariodetails/ScenarioWithDetails.scala
#	designer/server/src/main/scala/pl/touk/nussknacker/ui/api/ProcessesResources.scala
#	designer/server/src/main/scala/pl/touk/nussknacker/ui/server/AkkaHttpBasedRouteProvider.scala
#	designer/server/src/test/scala/pl/touk/nussknacker/test/base/it/NuResourcesTest.scala
@TouK TouK deleted a comment from github-actions bot Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant