Feature/display authors names from git repo

[ammardev]

Hello there,

This is a draft PR for getting information about each individual page author.
Data about authors are fetched from the GIT repository.

It's an improved version of "Added/updated by - from git" in Future Development issue #6.

These are some examples of the final result:

• A page with one contributor:
  

• A page with many contributors:
  

The design is inspired by vs code feature:

This update needs some refactoring and work. But I wanted to get feedback from the repo owner and the community before implementing the whole feature. So, please feel free to comment if you have any ideas or improvements for this feature.

The remaining tasks to complete this draft:

• Backend tasks
  • Extract all git-related logic to a separate class.
  • Check if git is installed before executing the needed commands.
  • Check if the file is committed in a repository. (Ignore if not committed)
  • Add configuration in the conf file to disable/enable git features.
  • Add tests.
• Frontend tasks
  • Expand the list of authors on click (Maybe I will expand it in a modal, Not sure yet)

[saleem-hadad]

This is very cool, are you planning to continue? @ammardev

[ammardev]

Yes, Of course.

I'm busy right now, But I would like to continue as soon as possible.

[saleem-hadad]

Hi @ammardev , I hope you're doing well.
any updates on this PR?

[ammardev]

Hi @saleem-hadad , I'm OK thanks.

I didn't have much time to contribute to open-source projects in the previous few months. But I'm planning to continue soon.
I'll try to finish this PR the next weekend

[ammardev]

Hello, @saleem-hadad

Sorry for being late.

My updates are finished for the backend side. I was planning to write some tests. But then I realized that I can't actually mock git operations (Commits). Do you have any idea for testing?

[saleem-hadad]

Yes, you can have an interface for GitService and then mock the interface in testing, similar to this logic
https://github.com/saleem-hadad/larecipe/blob/master/tests/Feature/InterchangeableMarkdownParserTest.php#L26