Skip to content
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.

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

Fix Issue #3075 by using None for local fields instead of Some(false) #3088

Merged
merged 5 commits into from Jun 14, 2023

Conversation

Neshura87
Copy link
Contributor

@Neshura87 Neshura87 commented Jun 14, 2023

Link to Issue: #3075

@Neshura87 Neshura87 changed the title Fix https://github.com/LemmyNet/lemmy/issues/3075 by using None for local fields instead of Some(false) Fix Issue #3075 by using None for local fields instead of Some(false) Jun 14, 2023
Neshura87 added a commit to Neshura87/lemmy that referenced this pull request Jun 14, 2023
@Nutomic Nutomic enabled auto-merge (squash) June 14, 2023 10:04
@Nutomic
Copy link
Member

Nutomic commented Jun 14, 2023

@dessalines comment_view::tests::test_crud comment_view::tests::test_languages keep failing randomly.

@Nutomic Nutomic merged commit 558035b into LemmyNet:main Jun 14, 2023
1 check passed
@Neshura87 Neshura87 deleted the patch-1 branch June 14, 2023 12:36
makotech222 pushed a commit to hexbear-collective/lemmy that referenced this pull request Jun 20, 2023
…e(false) (LemmyNet#3088)

* Set insert and update form "local" to None

* Extra space to retrigger woodpecker

* Resolve code review request

LemmyNet#3088 (comment)

* Remove extra space after None,
@snowe2010
Copy link

Did this fix #3198 ?

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.

None yet

3 participants