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 'main' into split_user_table

Adding some comments to notifs.

Moving send email check inside function.

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

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.

2nd pass. JWT now uses local_user_id

fixing tests 3

Fixing some tests 2

Fixing some tests

A first pass.

~80% done

Add memory limit for pictrs

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

Merge pull request #1442 from LemmyNet/fix_deploy_version_1

Fixing deploy version.

Fixing deploy version.

Adding 0.9.8 release notes.

Version 0.9.8

Order outbox by published, not id

Merge remote-tracking branch 'yerba/main'

Merge pull request 'Specify order for activities query (fixes #1436)' (#169) from outbox-order into main

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

Fix aggregates time columns 2 (#1427)

* Adding a new comment sort. Fixes #1294

* Fixing a migration comment.

* Adding a comment for newest_comment_time_necro

* Make sure federated items set correct aggregates fields in trigger.

- Fixes #1402

Adding a new comment sort for posts. Fixes #1294 (#1425)

* Adding a new comment sort. Fixes #1294

* Fixing a migration comment.

* Adding a comment for newest_comment_time_necro

Fix new compiler warning

Specify order for activities query (fixes #1436)

Merge pull request #1435 from LemmyNet/fix_lemmy_prod_open_port

Closing open lemmy-ui prod port. Fixes #1430

Closing open lemmy-ui prod port. Fixes #1430

Adding a few more tables.

Starting on user_ to person migration.

Merge remote-tracking branch 'origin/main'

Merge pull request 'Hide followed communities, except for own user (fixes #1303)' (#168) from hide-followed into main

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

Hide followed communities, except for own user (fixes #1303)

Merge pull request #1426 from LemmyNet/rss_link_post

Change RSS feeds to use lemmy URL for the rss link. Fixes #1378

Change RSS feeds to use lemmy URL for the rss link. Fixes #1378

Merge remote-tracking branch 'origin/main'

Merge pull request 'Explicitly mark posts and comments as public (ref #1220)' (#167) from comments-posts-public into main

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

Merge pull request #1422 from LemmyNet/display_name_limit

Display name limit