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

Update template URL for contributor's commit list for gitlab (dynamically adjusting for correct branch) #129

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdlutz24
Copy link

@mdlutz24 mdlutz24 commented Aug 1, 2019

Currently clicking on the number of issues takes you to the non-function pre-gitlab link.

This updates to a gitlab link, although we would have to update the head branch each minor.

@lauriii
Copy link
Owner

lauriii commented Aug 19, 2019

Sorry for the delayed feedback! I was on vacation for 2 weeks. We should try to find a way to automatically change the URL since it would be hard to keep this up to date. I would prefer removing the URL altogether temporarily given that it isn't working until we can figure out a way to create this link without requiring manual work.

@mdlutz24
Copy link
Author

mdlutz24 commented Aug 19, 2019

I do agree, and not being a ruby developer, I didn't really try. I believe in json.rb we could parse the output of 'git remote set-head origin -a' or 'git remote show origin' to get the current HEAD branch, save that to data.json as a top level key and insert it into the template in cores.rb, but I will have to look up the syntax and methods needed to do that. Before I try, it does bring up an interesting question.

What do you want to do with this site when the D9 branch opens in a month or two?

Drop everyone to zero, and start counting again from the branch? Have separate lists for D8 and D9? Just keep going and rebrand slightly to keep track of contributions to drupal 8+?

@mdlutz24 mdlutz24 changed the title Adjust template URL Update template URL for contributor's commit list for gitlab (dynamically adjusting for correct branch) Aug 26, 2019
@mdlutz24
Copy link
Author

Fair warning, this is the first Ruby code I've ever messed with, but I think this should do the trick to dynamically pick the correct branch. The questions regarding D9 strategy above are really unrelated and affect the whole counting algorithm and are way out of scope here. This just links to whichever branch we are counting from, which is the right thing to do.

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

Successfully merging this pull request may close these issues.

2 participants