[about] Cannot return null for non-nullable field About.topicCount.

biohackinginternational commented 6 months ago

Describe the bug

A clear and concise description of what the bug is. When using the application and clicking "Latest" or "Top"(only those), a new line shows in the output of the prose instance saying "[about] Cannot return null for non-nullable field About.topicCount."

REQUIRED: Include the contents of `api/.env` and `frontend/.env`

If you are manually overriding any environment variables when running the package scripts, include those as well.

api/.env

PROSE_DISCOURSE_HOST=https://forum.biohackinginternational.com

frontend/.env

No content

REQUIRED: answer the following questions:

If this is related to your mobile app:
- Are you running it through Expo, or did you build it? Built it
- Is it running on your mobile device or on a simulator? Mobile Device, Same issue in simulator

To Reproduce

Steps to reproduce the behavior:

Go to testflight
Click on install
Click on open
Tap "Latest" or "Top"
"[about] Cannot return null for non-nullable field About.topicCount." pops up in the output of the prose

Expected behavior

Upon clicking "Latest" or "Top" nothing should appear in the console.

Screenshots

If applicable, add screenshots to help explain your problem.

Environments

Desktop (please complete the following information):

OS: [e.g. iOS] iOS
Browser [e.g. chrome, safari]
Version [e.g. 22] 17.0

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

GraphQL API (please complete the following information):

Please indicate where the Prose GraphQL API is running.

https://prose.biohackinginternational.com

Are you only running it locally on your development machine? Production

Have you deployed it somewhere? If so, what domain or IP address did you deploy it at? https://prose.biohackinginternational.com

How specifically did you configure the mobile app to connect to the API? Set proseUrl for everything in frontend/Config.ts to https://prose.biohackinginternational.com

Additional context

Add any other context about the problem here. All the content and stuff shows but still has those output in the logs, is this possibly tied with the notification issue?

starjustice commented 6 months ago

Hey @biohackinginternational, thanks for reporting this bug. The fix for this issue will be included in the next release. However, if you'd like to try fixing it yourself, you can navigate to the file api/src/resolvers/site/aboutQuery.ts.

Modify the code on line 14 to the following:

    let {
      data: {
        about: {
          stats: {
            topics_count: topicsCount,
            topic_count: topicCount,
            posts_count: postsCount,
            post_count: postCount,
          },
        },
      },
    } = await context.client.get(siteUrl);

    return {
      topicCount: topicCount || topicsCount,
      postCount: postCount || postsCount,
    };

Kinostrome commented 3 months ago

Hey @biohackinginternational, since it has been several months since we have heard from you about this issue, we will soon close it as resolved.

Please let us know if you have further questions.

Kinostrome commented 3 months ago

Closing this since we never heard back, and the issue should already be fixed in the latest release.

lexiconhq / lexicon