Add profile stories and community feeds - #983
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0eeab6b435
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| ) | ||
| page = Paginator(selections, 10).get_page(page_number) | ||
| for selection in page.object_list: | ||
| selection.display_description = selection.description |
There was a problem hiding this comment.
When an admin hides a PersonProfileTag description, the personal profile path blanks it, but the community skill feed still copies selection.description directly into display_description; because community/_skill_feed_item.html renders that field through _story_display.html, any hidden interest/skill text remains publicly visible on /community/skills/. Hidden profile text should be filtered the same way as profile pages before it is placed in the public feed context.
AGENTS.md reference: AGENTS.md:L969-L973
Useful? React with 👍 / 👎.
| raise ValidationError(f"每个标签最多保留 {MAX_STORY_IMAGES} 张图片。") | ||
| to_delete.delete() | ||
|
|
||
| if locked_selection.description != description: |
There was a problem hiding this comment.
If a user only adds or deletes images while leaving the description unchanged, this block is skipped, so the PersonProfileTag.updated_at field is never saved even though the image set changed. Since the community skill feed is ordered by -updated_at, image-only uploads or removals can stay buried at their old tag-selection time instead of appearing as the newest profile story update.
Useful? React with 👍 / 👎.
|
建议补充以下测试： |
2 participants
Summary
Database changes
Includes app migrations 0017 through 0021 for profile tags, stories, images, likes, and comments.
Testing
python manage.py test app.test.test_profile_tags(30 passed)python manage.py test(243 passed)python manage.py makemigrations --check --dry-rungit diff --check