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

seleniumFix: fixed naming of xpath variables, added missing property to ... #69

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@srse
Contributor

srse commented Jan 23, 2013

...gitblit.properties and test-ui-gitblit.properties, changed naming of methods (according to decision that owners is a much shorter word than repository administrator and that owners is fits better if you think of collective responsibilities and collective ownership)

SHaselbauer
seleniumFix: fixed naming of xpath variables, added missing property …
…to gitblit.properties and test-ui-gitblit.properties, changed naming of methods (according to decision that owners is a much shorter word than repository administrator and that owners is fits better if you think of collective responsibilities and collective ownership)
@@ -146,6 +146,13 @@ git.defaultAccessRestriction = NONE
# SINCE 1.1.0
git.defaultAuthorizationControl = NAMED
# The default handling of branch indexing.

This comment has been minimized.

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

logger.info("Lucene executor is scheduled to process indexed branches every 2 minutes.");
scheduledExecutor.scheduleAtFixedRate(luceneExecutor, 1, 2, TimeUnit.MINUTES);
boolean branchIndexingActivated = settings.getBoolean(Keys.git.branchIndexingActivated, true);

This comment has been minimized.

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

@@ -130,6 +130,13 @@ git.defaultAccessRestriction = NONE
# SINCE 1.1.0
git.defaultAuthorizationControl = NAMED
# The default handling of branch indexing.

This comment has been minimized.

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

@gitblit

gitblit Jan 23, 2013

Owner

I specifically excluded this because it is redundant.
see web.allowLuceneIndexing

@gitblit

This comment has been minimized.

Show comment
Hide comment
@gitblit

gitblit Jan 30, 2013

Owner

Hi Sarah, have I already merged the important changes in this request by pulling a different request?

Owner

gitblit commented Jan 30, 2013

Hi Sarah, have I already merged the important changes in this request by pulling a different request?

@srse

This comment has been minimized.

Show comment
Hide comment
@srse

srse Jan 30, 2013

Contributor

Hi James,
right, it's already merged by pull request #70. So, we can close this one as well.

Contributor

srse commented Jan 30, 2013

Hi James,
right, it's already merged by pull request #70. So, we can close this one as well.

@srse srse closed this Jan 30, 2013

gitblit added a commit that referenced this pull request May 9, 2014

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