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

jbossdemocentral string is hard-coded so forks don't work #83

Open
trepel opened this issue Sep 20, 2017 · 2 comments
Open

jbossdemocentral string is hard-coded so forks don't work #83

trepel opened this issue Sep 20, 2017 · 2 comments

Comments

@trepel
Copy link

trepel commented Sep 20, 2017

The jbossdemocentral string is hard-coded in several places accross the code base. It makes the forks of the repo not work as expected.

It is mostly hard-coded in various openshift templates (yaml/json files). One ocurrence is also in coolstore-ui/package.json (not causing issues) and in https://github.com/jbossdemocentral/coolstore-microservice/blob/1.0.x/openshift/scripts/provision-demo.sh#L327 (which I think should be fixed).

@siamaksade
Copy link
Collaborator

The provision-script.sh uses the env var GITHUB_ACCOUNT to refer to other repos in case it is forked. The reference you mentioned is a sed command and does not interfere with forks.

What issue did you run into?

@trepel
Copy link
Author

trepel commented Oct 4, 2017

Hi @siamaksade, thanks for reply. To be honest I can't remember. Thinking about it again it very well might be that I thought the root cause of issue #77 was caused by jbossdemocentral being hard-coded which was not true.

If you or someone else from coolstore developers went through all the jbossdemocentral ocurrences in code base and all occurences seem ok to you, I am not going to argue. Feel free to close this one. I will reprovision from fork and if I encounter something I will create another more specific issue.

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

No branches or pull requests

2 participants