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

Only run profile js on pages that need it #8120

Merged
merged 1 commit into from Oct 27, 2014

Conversation

4 participants
@cirosantilli
Contributor

cirosantilli commented Oct 21, 2014

Needed to split up the .profile-groups-avatars line into User since it does not affect profile, only user. It's misnamed.

Once again $ -> is evil.

@TeatroIO

This comment has been minimized.

TeatroIO commented Oct 21, 2014

I've prepared a stage. Click to open.

@Razer6 Razer6 added this to the 7.5 milestone Oct 23, 2014

dzaporozhets added a commit that referenced this pull request Oct 27, 2014

Merge pull request #8120 from cirosantilli/profile-js-only-needed
Only run profile js on pages that need it

@dzaporozhets dzaporozhets merged commit 01441f5 into gitlabhq:master Oct 27, 2014

1 check passed

default The build passed on Semaphore.
Details

@cirosantilli cirosantilli deleted the cirosantilli:profile-js-only-needed branch Oct 27, 2014

tigefa4u pushed a commit to tigefa4u/gitlabhq that referenced this pull request Nov 6, 2018

Merge branch '7126-batch-operations' into 'master'
Geo: Admin > Geo > Projects support for batch operations

Closes gitlabhq#8120

See merge request gitlab-org/gitlab-ee!7806
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment