maximelouet / blih-web

Web interface for BLIH, the EPITECH repository management tool.
https://blih.saumon.io
Apache License 2.0
46 stars 6 forks source link

Added a git url to git view #9

Closed ImOverlord closed 5 years ago

ImOverlord commented 5 years ago

Description

Adds Repository URL on the repository edit view

screenshot from 2019-02-07 11-22-25

Adds Repository URL on the repository creation view, the URL updates as you type the repository name.

screenshot from 2019-02-07 11-22-34

Type of change

Please delete options that are not relevant.

ImOverlord commented 5 years ago

Do you have an ETA for when you will be able to review this PR, and maybe accept it?

maximelouet commented 5 years ago

Hello! Thank you for your contribution.

However, that kind of information is not provided by the BLIH API. The BLIH Web philosophy is to only show data retrieved from the BLIH API. Other informations that can be inferred from this data (such as the repository URL) do not belong to the BLIH Web interface and can be recreated via very simple scripts.

If Epitech students are not able to write a repository URI properly, it is not our job to do it for them. Epitech encourages autonomous learning ; I agree with this philosophy. Students should learn some shell scripting or understand how an SSH URI works.

I appreciate your work and the time you took to understand and extend my code, but sadly I won't accept your pull request because the data it shows is useless to me, and thus clutters the BLIH Web interface.

Lastly, I don't appreciate trolling and spam on this repository, that's why I took some time to respond to the pull request.