Skip to content

Test #4701

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Apr 28, 2025
Merged

Test #4701

merged 6 commits into from
Apr 28, 2025

Conversation

dreamer6680
Copy link
Contributor

No description provided.

@gru-agent Gru Agent
Copy link
Contributor

gru-agent bot commented Apr 28, 2025 •
edited
Loading

TestGru Assignment

Summary

Link CommitId Status Reason
Detail 96a14c9 ✅ Finished

Files

File Pull Request
projects/app/src/web/context/useInitApp.ts ❌ Failure (I failed to write the unit tests for the file.)
projects/app/src/web/support/marketing/utils.ts 🔴 Closed #4702

Tip

You can @gru-agent and leave your feedback. TestGru will make adjustments based on your input

@github-actions GitHub Actions
Copy link

Preview mcp_server Image: ghcr.io/labring/fastgpt-pr:fatsgpt_mcp_server_96a14c9da3e369568892297f141c76d2884d0c43

@github-actions GitHub Actions
Copy link

Preview sandbox Image: ghcr.io/labring/fastgpt-pr:fatsgpt_sandbox_96a14c9da3e369568892297f141c76d2884d0c43

@github-actions GitHub Actions
Copy link

Preview fastgpt Image: ghcr.io/labring/fastgpt-pr:fatsgpt_96a14c9da3e369568892297f141c76d2884d0c43

@gru-agent gru-agent bot mentioned this pull request Apr 28, 2025
Closed
c121914yu
c121914yu reviewed Apr 28, 2025
View reviewed changes
c121914yu
c121914yu reviewed Apr 28, 2025
View reviewed changes
projects/app/src/web/context/useInitApp.ts
...(utm_content && { shortUrlContent: utm_content })
};
setUtmParams(utmParams);
if (utm_workflow) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

不可能为空呀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

可能啊，有的推文是没有json的

@dreamer6680
Optimized the JsonImportModal shutdown logic to ensure that UTM param…
55f64ee 
…eters and workflows are removed when closing.
@github-actions GitHub Actions
Copy link

Preview mcp_server Image: ghcr.io/labring/fastgpt-pr:fatsgpt_mcp_server_55f64ee7161c38a7dbfe960857548758d3688408

@github-actions GitHub Actions
Copy link

Preview sandbox Image: ghcr.io/labring/fastgpt-pr:fatsgpt_sandbox_55f64ee7161c38a7dbfe960857548758d3688408

@github-actions GitHub Actions
Copy link

Preview mcp_server Image: ghcr.io/labring/fastgpt-pr:fatsgpt_mcp_server_c77286ed6f04427178c6880df67dafc9f9324d43

@github-actions GitHub Actions
Copy link

Preview sandbox Image: ghcr.io/labring/fastgpt-pr:fatsgpt_sandbox_c77286ed6f04427178c6880df67dafc9f9324d43

@github-actions GitHub Actions
Copy link

Preview fastgpt Image: ghcr.io/labring/fastgpt-pr:fatsgpt_55f64ee7161c38a7dbfe960857548758d3688408

@github-actions GitHub Actions
Copy link

Preview fastgpt Image: ghcr.io/labring/fastgpt-pr:fatsgpt_c77286ed6f04427178c6880df67dafc9f9324d43

@c121914yu c121914yu merged commit e0b85ca into labring:main Apr 28, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@c121914yu c121914yu c121914yu approved these changes

Assignees
No one assigned
Labels
size/S
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dreamer6680 @c121914yu