Address review comments

Fix conflicts in docker-compose.yml

Merge branch 'main' into federated-moderation

Merge pull request #1499 from LemmyNet/strictly_type_db_ids

Strictly type db ids

Merge pull request #1428 from LemmyNet/split_user_table

Split user table

Merge branch 'split_user_table' into strictly_type_db_ids

Merge branch 'main' into split_user_table

Set CARGO_HOME for CI so deps arent redownloaded (#1497)

* Set CARGO_HOME for CI so deps arent redownloaded

* run find on x86

* fix path

* cleanup

* try again

* use mv

Merge branch 'split_user_table' into strictly_type_db_ids

Merge branch 'main' into split_user_table

Strictly typing DB id fields. Fixes #1498

Fix three federation test cases

Adding some comments to notifs.

Moving send email check inside function.

Refactor activitypub code

For FromApub trait, use `is_mod_action: bool` instead

Allow remot moderators to do Remove/Post and Remove/Comment

Merge pull request 'Insert announced activities into DB for fetching (fixes #1494)' (#187) from insert-local-announce into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/187

Removing some TODOS.

Version 0.10.0-rc.5

Some changes

- Changing claim name to local_user_id to facilitate logout.
- Changing AddAdmin back to using person_id

Insert announced activities into DB for fetching (fixes #1494)

Merge pull request #1491 from LemmyNet/upgrade_pictrs_3

Upgrading pictrs.

Only sending private message if its a local user.

Trying to fix API tests.

Change joinuser, sendusermessage to use local_user_id

Fix code to allow sticky/lock from remote moderators

Merge pull request 'Add memory limit for pictrs' (#186) from nutomic-patch-1 into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/186

Upgrading pictrs.

WIP: check that modifications are made by same user, add docs

2nd pass. JWT now uses local_user_id

Fix clippy warning

Add check so only mods can change stickied/locked state of posts

fixing tests 3

Fixing some tests 2

Add check so only author or mods can edit posts/comments

Fixing some tests

Allow for remote mods to remove posts/comments

A first pass.

~80% done

Add memory limit for pictrs

Allow adding remote users as community mods (ref #1061)

Update activitystreams to 0.7.0-alpha.11

Implemented receiving activities to add/remove remote mods

Move moderators collection to separate HTTP endpoint

Use collection for moderators, instead of `attributedTo` (ref #1061)

Merge pull request #1485 from PatMulligan/fix-docker-compose-yaml

revert Compose file version from 3.3 to 2.2

revert Compose file version from 3.3 to 2.2

Adding more mem limits

bump memory limit of iframely

Merge pull request #1478 from LemmyNet/fix_wrong_urls

Fixing wrong user_ and community icon and banner urls.

Merge pull request #1479 from LemmyNet/fix_extra_categories

Remove extra category_id s . Fixes #1429

Remove extra category_id s . Fixes #1429

Fixing wrong user_ and community icon and banner urls.

- Fixes #1477

Merge branch 'main' of https://github.com/lemmynet/lemmy

Merge pull request 'Forbid usage of unwrap' (#179) from clippy-unwrap into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/179

Merge pull request 'Remove category from activitypub context' (#183) from context-remove-category into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/183

Merge pull request 'Update test script' (#182) from update-test-script into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/182

Remove category from activitypub context

Adding a password length check to other API actions. (#1474)

* Adding a password length check to other API actions.

- Fixes #1473

* Fixing comment.

Update test script

Use URL type in most outstanding struct fields (#1468)

* Use URL type in most outstanding struct fields

This fixes all known remaining cases where url fields are stored as
plain strings, with the exception of form fields where empty strings
are used as sentinels (see `diesel_option_overwrite_to_url`).

Tested for regressions in the federated docker setup attempting to
exercise all changed fields, including through apub federation.

Fixes #1385

* Add migration to fix blank-string post.url values to be null

This also then fixes #602

* Address review feedback

- Fixed some unwraps and err message formatting
- Bumped the `url` library to 2.2.1 to fix a bug with serde error
  messages
- Add unit tests for the two diesel option override functions
- Fix migration teardown by adding a no-op

* Rename lemmy_db_queries::Url to lemmy_db_queries::DbUrl

* fix compile error

* box PostOrComment variants

Forbid usage of unwrap

Merge pull request 'Upgrade Rust version' (#181) from upgrade-rust into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/181

Upgrade Rust version

Rewrite settings implementation. Fixes #1270 (#1433)

* A first attempt at using deser-hjson. Fixes #1270

* Trying to fix tests, try 1

* Trying to fix tests, try 2

* A few fixes to deser_hjson

- Removing unwrap_or_defaults, using impl functions.
- Reorganized settings

* Make clippy happy

* hjson list strings must be quoted.

* Adding support for env vars.

* Moving to structs and defaults file.

* Moving settings default and struct.

Merge pull request 'Rename `lemmy_structs` to `lemmy_api_structs`' (#180) from rename-structs into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/180

Rename `lemmy_structs` to `lemmy_api_structs`

Merge pull request #1465 from arjenpdevries/patch-2

Update README.md

Update README.md

Federation is probably more complete than suggested here.

In response to:
https://github.com/LemmyNet/lemmy/issues/647#issuecomment-787068478

Still continuing on....

Merge pull request 'Remove federation backward compatibility code (ref #1220)' (#164) from remove-backwards-compatibility into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/164

Merge pull request 'Remove code for apub compatibility with Lemmy v0.8.9 and older' (#178) from remove-apub-compat-code into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/178

Remove federation backward compatibility code (ref #1220)

Remove code for apub compatibility with Lemmy v0.8.9 and older

Merge pull request #1455 from ajyoon/fix-flaky-db-tests

Support plain `cargo test` and disable unused doctests for speed

Support plain `cargo test` and disable unused doctests for speed

Since DB tests execute diesel migrations automatically, concurrent
execution causes flaky failures from simultaneous migrations. This can
be worked around using `cargo test --workspace -- --test-threads=1`,
which is what the CI config does, but this is not intuitive for
newcomer developers and unnecessarily slows down the test suite for
the majority of tests which are safe to run concurrently. This fixes
this issue by integrating with the small test crate `serial_test` and
using it to explicitly mark DB tests to run sequentially while
allowing all other tests to run in parallel.

Additionally, this greatly improves the speed of `cargo test` by
disabling doc-tests in all crates, since these are aren't currently
used and cargo's doc-test pass, even when no doc-tests exist, has
significant overhead. On my machine, this change significantly
improves test suite times by about 85%, making it much more practical
to develop with tools like `cargo watch` auto-running tests.

Done with user->person migrations, now to code.

Merge branch 'main' into split_user_table

Merge branch 'main' into remove-integration-tests

Merge remote-tracking branch 'yerba/main'

Merge pull request 'Remove categories (fixes #1429)' (#176) from remove-categories into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/176

Remove broken actix_rt test

Remove integration tests (fixes #1449)

Merge pull request #1451 from LemmyNet/update_cargo_chef

Use more recent version of cargo chef.

In remove categories down migration, add default for category

Remove categories (fixes #1429)

Merge pull request 'Better type safety for activity parsing' (#175) from apub-type-safety into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/175

Use more recent version of cargo chef.

Better type safety for activity parsing

Merge pull request 'Dont include community in comment `to` field (fixes #1446)' (#174) from no-community-in-comment-to into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/174

Fix lemmy_dev ansible playbook

Dont include community in comment `to` field (fixes #1446)

Merge pull request 'Order outbox by published, not id' (#171) from outbox-order-published into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/171

Merge pull request 'Use `name` field for post titles instead of `summary` (ref #1220)' (#173) from apub-post-name into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/173

Merge pull request 'Fix clippy error upper_case_acronyms' (#172) from fix-clippy into main

Reviewed-on: https://yerbamate.ml/LemmyNet/lemmy/pulls/172

Use `name` field for post titles instead of `summary` (ref #1220)

Fix clippy error upper_case_acronyms

Updating release version.

Version 0.9.9