fixed fetch events bug

[Azad99-9]

What kind of change does this PR introduce?

This PR fixes the event fetching bug in explore events page.

Issue Number:

Fixes #2307

Did you add tests for your changes?

Snapshots/Videos:

If relevant, did you update the documentation?

Summary

Removed a field name registrants from the query which is not actually defined for the Collection Event in the database.

Does this PR introduce a breaking change?

No

Other information

Have you read the contributing guide?
Yes

[github-actions]

Our Pull Request Approval Process

We have these basic policies to make the approval process smoother for our volunteer team.

Testing Your Code

Please make sure your code passes all tests. Our test code coverage system will fail if either of these two conditions occur:

1. The overall code coverage drops below the target threshold of the repository
2. Any file in the pull request has code coverage levels below the repository threshold

The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing.

Reviewers

When your PR has been assigned reviewers contact them to get your code reviewed and approved via:

1. comments in this PR or
2. our slack channel

Reviewing Your Code

Your reviewer(s) will have the following roles:

1. arbitrators of future discussions with other contributors about the validity of your changes
2. point of contact for evaluating the validity of your work
3. person who verifies matching issues by others that should be closed.
4. person who gives general guidance in fixing your tests

Other

🎯 Please be considerate of our volunteers' time. Contacting the person who assigned the reviewers is not advised unless they ask for your input. Do not @ the person who did the assignment otherwise.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (6d95344) 92.37% compared to head (7eb0e54) 92.37%.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2308   +/-   ##
========================================
  Coverage    92.37%   92.37%           
========================================
  Files          158      158           
  Lines         7632     7632           
========================================
  Hits          7050     7050           
  Misses         582      582           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Azad99-9]

@noman2002 @Cioppolo14 @Kevoniat please assign reviewers for this PR.

[noman2002]

Looks good.

[Dante291]

@palisadoes @Azad99-9 @noman2002 This PR did solve the problem of Events not getting fetched but with the updated FetchOrgEvents Query we are not fetching information about attendees/registrants for the event.

That is this will always show zero even if there are registrants for the event

Also there is another bug in event info page where attendees are not getting fetched too

so I'm going to make a separate issue for this and work on it.

[Azad99-9]

@Dante291 please check there is no field called attendees/registrants in Event Collection. If it's there then definitely you can.

[Azad99-9]

@Dante291 I suggest, you can raise an issue in Talawa API asking for addition of fields

1. Registrants.
2. Event Image.

and any other if required to the Event Collection and then work on the previously mentioned issue.

So that not only the registrants but also the event Image bug also will be fixed. thanks.

[Dante291]

@Dante291 I suggest, you can raise an issue in Talawa API asking for addition of fields

1. Registrants.
2. Event Image.

and any other if required to the Event Collection and then work on the previously mentioned issue.

So that not only the registrants but also the event Image bug also will be fixed. thanks.

@Azad99-9 I think registrants and attendees are basically same (I cross checked with admin portal too) so there is no requirement for registrant field for as we could fetch attendees by modifying the query.

about the event image part @palisadoes @noman2002 could you clarify whether it is needed or not?

[palisadoes]

1. Registrants are people who have expressed an interest in attending an event. This is useful when the event is housed in a venue with limited capacity.
2. Attendees are those people who actually attended the event

If that distinction isn't present it will need to be.

[Dante291]

1. Registrants are people who have expressed an interest in attending an event. This is useful when the event is housed in a venue with limited capacity.
2. Attendees are those people who actually attended the event

If that distinction isn't present it will need to be.

@palisadoes Currently we do not have any field like registrants in Event model of API, though there is query for fetching registrantsByEvent but it is not fetching registrants properly.

Also on admin portal we are considering registrants as attendees, and displaying data according to that I also checked there is no query like registrantsByEvent in Talawa-Admin to fetch registrants.

[palisadoes]

I have created base issues for this.

• Add Support for Event Invitees and Registrants talawa-api#1644
• Add Support for Event Images talawa-api#1645

What other issues will be necessary?