-
Notifications
You must be signed in to change notification settings - Fork 84
Added CENX company as a user #51
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
Added CENX company as a user #51
Conversation
Please don't merge until I confirm the image with my company. |
Merging base master branch back into my fork
@Tattoo @pekkaklarck |
Hi @bkhouri , Few comments/questions: Why is the logo for different thing than the description? The company is CENX, but the logo is "cortx"? The logo does not follow the sizes of other logos ie. max height 68px, making it stand out on the page. I'm afraid the description is also too wordy for the layout. Frankly, this bit:
... does not really have anything to do with how you are using Robot Framework. Maybe this could be removed from the description text, thus shortening it? The entries (besides the first one) should be in alphabetical order, meaning currently this entry is one off of the correct place: it should come after Condé Nast but before Ficosa. Do you want to implement based on my suggestion, or should I do my version of how the entry in the list could look? |
Hi, |
@rsmparthiban are you @bkhouri or from CENX? |
@rsmparthiban You'd like to add a new organization as a user, right? Please submit a new issue about it. |
@Tattoo I'm from CENX. I know the image does not show CENX, but that's the image the marketing team wants to use - don't ask me why. cortx is the product. CENX is the company. For the text, used whatever the marketing team wanted. Feel free to update the PR with your revisions. |
@bkhouri , I made the changes detailed in my comment. If you want to work on them more, please open a new issue. Thanks for contributing! |
Got approval to add my company, CENX, listed under the robot framework users.
fixes #50