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

Improved NestBot /events command #657

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

abhayymishraa
Copy link
Collaborator

Resolves #641

Add the PR description here.
Task Completed:

  • Improved /events slack command
  • Added get_events_data() in utils to fetch data from the source file
  • Added testcases for it (covers 96%)

Preview/ScreenShots:

Slack:

Screenshot from 2025-01-29 17-31-28
Screenshot from 2025-01-29 17-31-55

TestcaseCoverage:

Screenshot from 2025-01-29 17-27-03

@abhayymishraa
Copy link
Collaborator Author

abhayymishraa commented Feb 4, 2025

After this commit:

  • Added command to populate database
  • Added graphql (Query/Node) for event model
  • Added NestBot command /events to fetch data from directly from graphql (internally)
  • Extended Event Model to include new changes
  • updated testcases as per new chnages
  • Sorted the events with the StartDate

Chnages Look Like:

Screenshot from 2025-02-05 01-43-06
Screenshot from 2025-02-05 01-43-26

@abhayymishraa
Copy link
Collaborator Author

  • Removed event sync logic as it's of no use

@abhayymishraa
Copy link
Collaborator Author

hey @arkid15r , i removed the syncing lofgic for removing any data once it will be removed from the data then we can remove the fields that were previously used by them

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve NestBot /events command
1 participant