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

Remove MAX_NAME_LEN from UiConstants #1698

Merged
merged 4 commits into from
Jul 25, 2017

Conversation

europ
Copy link
Member

@europ europ commented Jul 17, 2017

Issue: #1661

We have removed MAX_NAME_LEN constant from UiConstants. MAX_NAME_LEN was moved to ViewHelper and also added ViewHelper prefix for each occurrence of it.

@miq-bot
Copy link
Member

miq-bot commented Jul 17, 2017

This pull request is not mergeable. Please rebase and repush.

@europ europ force-pushed the remove-ui-constants branch from ed8d161 to 11fc0a3 Compare July 17, 2017 13:23
@chessbyte chessbyte changed the title Remove ui constants Remove MAX_NAME_LEN from UiConstants Jul 17, 2017
@martinpovolny
Copy link
Member

MAX_NAME_LEN = 20 # Default maximum name length

move the comment as well please

@europ europ force-pushed the remove-ui-constants branch from 3fa6709 to 11fc0a3 Compare July 18, 2017 09:00
@martinpovolny
Copy link
Member

Restarting travis.

@martinpovolny
Copy link
Member

Please, rebase.

@miq-bot
Copy link
Member

miq-bot commented Jul 20, 2017

This pull request is not mergeable. Please rebase and repush.

@martinpovolny
Copy link
Member

Restarted travis.

@martinpovolny
Copy link
Member

Please, rebase.

@miq-bot
Copy link
Member

miq-bot commented Jul 25, 2017

This pull request is not mergeable. Please rebase and repush.

@europ europ force-pushed the remove-ui-constants branch from c2ba549 to b005e8d Compare July 25, 2017 14:51
@europ europ force-pushed the remove-ui-constants branch from b005e8d to 78b3e35 Compare July 25, 2017 15:30
@europ europ force-pushed the remove-ui-constants branch from 78b3e35 to 06bcd38 Compare July 25, 2017 18:18
@miq-bot
Copy link
Member

miq-bot commented Jul 25, 2017

Checked commits europ/manageiq-ui-classic@2ab52ba~...06bcd38 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
32 files checked, 0 offenses detected
Everything looks fine. 🍰

@martinpovolny martinpovolny merged commit 70559e9 into ManageIQ:master Jul 25, 2017
@martinpovolny martinpovolny added this to the Sprint 66 Ending Aug 7, 2017 milestone Jul 25, 2017
@europ europ deleted the remove-ui-constants branch August 7, 2017 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants