Skip to content

fix:message api doc#15568

Merged
crazywoola merged 1 commit intolanggenius:mainfrom
marsDes:fix/messages-api-docs
Mar 12, 2025
Merged

fix:message api doc#15568
crazywoola merged 1 commit intolanggenius:mainfrom
marsDes:fix/messages-api-docs

Conversation

@marsDes
Copy link
Contributor

@marsDes marsDes commented Mar 12, 2025

Summary

when use Get Conversation History Messages api (v1/messages of Agent Assistant) it return

...
"agent thoughts":[
  {
    ...
   "message_files":[
     {
        "id":null
      }
   ]
    ...
   }
]
...

but the api doc show

 "message_files": [
   "976990d2-5294-47e6-8f14-7356ba9d2d76"
  ]

and /api/messages show

"files": [
   "976990d2-5294-47e6-8f14-7356ba9d2d76"
  ]

I think it's necessary to maintain unity

Tip

Close issue syntax: fix #15070

Screenshots

Before After
79bd8b4913807fffc3df6341654c62c 140fd1bd749c28b5a2f6d1f57a72779

Checklist

Important

Please review the checklist below before submitting your pull request.

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. 📚 documentation Improvements or additions to documentation labels Mar 12, 2025
Copy link
Member

@crazywoola crazywoola left a comment

Choose a reason for hiding this comment

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

LGTM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 12, 2025
@crazywoola crazywoola merged commit 1fab02c into langgenius:main Mar 12, 2025
7 checks passed
Scorpion1221 added a commit to yybht155/dify that referenced this pull request Mar 12, 2025
* commit '5019547d33eefa870786e5f981907ff2314a639a': (128 commits)
  fix: can not test custom tool (langgenius#15606)
  fix: no attribute error (langgenius#15597)
  fix: trace return null cause page crash (langgenius#15588)
  fix: set marketplace feature to false in feature_service.py (langgenius#15578)
  feat: env MAX_TOOLS_NUM (langgenius#15431)
  chore: use TenantAccountRole instead of TenantAccountJoinRole (langgenius#15514)
  fix: dataset editor (langgenius#15218)
  fix:message api doc (langgenius#15568)
  chore: remove unused parameter (langgenius#15558)
  feat: add debounced enter key submission to install form (langgenius#15445) (langgenius#15542)
  fix: invoke_error is not callable (langgenius#15555)
  fix: workflow loop node break conditions (langgenius#15549)
  fix: remove size prop in PlanBadge component because UpgradeBtn size … (langgenius#15544)
  feat: add click-away and mounting logic to agent setting component (langgenius#15521)
  chore: update opendal version (langgenius#14343)
  fix: nesting of conditional branches causing streaming output error (langgenius#14065)
  fix: update version to 1.0.1 in configuration and Docker files (langgenius#15478)
  fix notion page display (langgenius#15508)
  support workspace billing info (langgenius#15510)
  fix: update placeholders in version info modal to indicate optional field (langgenius#15499)
  ...

# Conflicts:
#	api/poetry.lock
#	web/pnpm-lock.yaml
parambharat pushed a commit to parambharat/dify that referenced this pull request Mar 31, 2025
Co-authored-by: mars <linjx2@by-health.com>
jsincorporated pushed a commit to jsincorporated/asaAi that referenced this pull request Jul 8, 2025
Co-authored-by: mars <linjx2@by-health.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📚 documentation Improvements or additions to documentation lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

the message api doc has wrong infomation

2 participants