Skip to content

Add GraphQL for commits over time#7

Draft
itsrealfake wants to merge 4 commits into
mainfrom
add-commits-over-time-for-author
Draft

Add GraphQL for commits over time#7
itsrealfake wants to merge 4 commits into
mainfrom
add-commits-over-time-for-author

Conversation

@itsrealfake

@itsrealfake itsrealfake commented May 20, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

query with ::

{
  commitsOverTimeForAuthor {
    commits {
     commitHash
      message
      summary
    }
   	date 
  }
}

itsrealfake
itsrealfake commented May 20, 2025
View reviewed changes
Comment thread docker-compose.yml
- "7687:7687" # Bolt protocol
environment:
- NEO4J_AUTH=neo4j/password # Set initial username/password # TODO Get this from ENV file
- NEO4JLABS_PLUGINS=["graph-data-science", "apoc"]

@itsrealfake itsrealfake May 20, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

not sure if we actually need graph-data-science

apoc gives us lots of cool stuff

itsrealfake
itsrealfake commented May 20, 2025
View reviewed changes
Comment thread backend/app/schema.py
commits_over_time_for_author = graphene.List(CommitsOnDate, description="List of Commits On Date")
def resolve_commits_over_time_for_author(self, info):
db = Neo4jDriver()
details = db.commits_over_time_for_actor("satoshin@gmx.com")

@itsrealfake itsrealfake May 20, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

need to fix this hardcoded satoshi.

i think it could take a query param list [{name, email}] and then do some WHERE name OR email are matched

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@itsrealfake