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.

Sign up for GitHub

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

Jump to bottom

Add GraphQL operations for club #67

Closed
wants to merge 15 commits into from
Closed

Add GraphQL operations for club #67

wants to merge 15 commits into from

Conversation

SriramPatibanda
Copy link
Member

Solved or Linked Issues

Fixes #58

Description and Changes

  • Add Club Type
  • Add Club Query
  • Add Club Mutation
  • Add Club Resolver

TODO: Error Handling, Search Clubs, Media need to be handled.

Work In Progress.

Your checklist for this pull request

🚨Please review the guidelines for contributing to this repository.

  • Make sure you are requesting to pull a work/topic/feature/bugfix branch (right side). Don't request your default branch!
  • Make sure you are making a pull request against the development (left side). Also you should start your branch off default branch and follow the branch naming guidelines.
  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.
  • I have added necessary documentation and mentioned the related issues (if appropriate)
  • I have added the appropriate labels, requested a review from a repository manager and assigned myself to this pull request.

Post merge checklist

@SriramPatibanda
Add Club Type
d896e54
@SriramPatibanda
Merges ClubSchema into graphql schemas
d74d07e
@SriramPatibanda
Adds facAd field to the model
c767692
@SriramPatibanda
Migrates ExecutiveType to a unique file
6e962a6
@SriramPatibanda
Adds club queries
0c28209
@SriramPatibanda
Adds updateClub and updateClubExecutive mutations
7897856
@SriramPatibanda
adds club schema
e0acda3
@SriramPatibanda
Adds empty resolvers
e6fdc74
@SriramPatibanda
grammer correction in HasPermission
b381267
@SriramPatibanda
Adds bsic documentation to club module
60d7963
@SriramPatibanda
Adds addClub mutation and getClub query
eb51c82 
Updates pm2, changes logo to ID type, sets required parameter of logo to false for development purposes, adds resolvers for addClub and getClub
@SriramPatibanda
Adds resolvers for listCubs and updateClub
09df254
@SriramPatibanda
Adds resolver for searchClubs. Changes updateClub resolver
3b320b9
@SriramPatibanda
updates required fields to non-null of club mutation
b6fbbf3
@SriramPatibanda
Adds deleteClub, changes updateClub resolver and fixes spelling errors
ffcae1f
@rutajdash rutajdash added the wip Work in Progress label May 4, 2021
@rutajdash rutajdash added this to the Phase 1 Launch milestone Oct 5, 2021
@rutajdash rutajdash removed this from the Phase 1 Launch milestone Oct 19, 2021
rutajdash
rutajdash reviewed Nov 18, 2022
View reviewed changes
Copy link
Member

@rutajdash rutajdash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@120EE0692
This requires a little more refinement and the addition of permission checks and media handling.

@rutajdash rutajdash closed this Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rutajdash rutajdash rutajdash left review comments

Assignees

@SriramPatibanda SriramPatibanda

Labels
wip Work in Progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Create Club Schema
2 participants
@SriramPatibanda @rutajdash