Skip to content

Finish adding post tags#5869

Merged
Nutomic merged 51 commits intomainfrom
add-post-tags
Jul 24, 2025
Merged

Finish adding post tags#5869
Nutomic merged 51 commits intomainfrom
add-post-tags

Conversation

@Nutomic
Copy link
Copy Markdown
Member

@Nutomic Nutomic commented Jul 17, 2025

  • adjust changes from previous PR: Community post tags (part 3/3: federation) #5636
  • remove lemmy: prefix from ap fields
  • code cleanup and query optimizations
  • rename display_name to name, cant be changed and is used to generate ap_id url
  • add http endpoint to fetch tags via activitypub
  • change api tests to use federation
  • new api endpoint PUT /api/v4/post/mod_update for mods to change tags and nsfw flag of posts
  • tag has new federation format which allows for adding instance tags or user tags later

@Nutomic Nutomic marked this pull request as ready for review July 21, 2025 12:53
@Nutomic
Copy link
Copy Markdown
Member Author

Nutomic commented Jul 23, 2025

All the comments should be addressed now.

Copy link
Copy Markdown
Member

@dessalines dessalines left a comment

Choose a reason for hiding this comment

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

Looks good, feel free to merge after you get CI passing.

.await?;

let mut post_form = PostUpdateForm {
nsfw: data.nsfw,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice, setting nsfw also.

Comment thread scripts/lint.sh Outdated

# Format sql files
find migrations crates/db_schema_setup/replaceable_schema -type f -name '*.sql' -print0 | xargs -0 -P 10 -L 10 pg_format -i
#find migrations crates/db_schema_setup/replaceable_schema -type f -name '*.sql' -print0 | xargs -0 -P 10 -L 10 pg_format -i
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Check.

Comment thread src/api_routes.rs
.route("/report/resolve", put().to(resolve_post_report))
.route("/notifications", post().to(update_post_notifications)),
.route("/notifications", post().to(update_post_notifications))
.route("/mod_update", put().to(mod_update_post)),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can't think of a better name, its pry fine.

@Nutomic Nutomic enabled auto-merge (squash) July 24, 2025 09:15
@Nutomic Nutomic merged commit 7dfd5ef into main Jul 24, 2025
2 checks passed
This was referenced Jul 24, 2025
@Nothing4You Nothing4You deleted the add-post-tags branch September 11, 2025 17:28
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