Skip to content

Comments

Bug 1384706, update screenshots for image catalog#3254

Merged
ahardin-rh merged 1 commit intoopenshift:masterfrom
ahardin-rh:UX-image-catalog
Nov 22, 2016
Merged

Bug 1384706, update screenshots for image catalog#3254
ahardin-rh merged 1 commit intoopenshift:masterfrom
ahardin-rh:UX-image-catalog

Conversation

@ahardin-rh
Copy link
Contributor

@jwforres
Copy link
Member

@ahardin-rh those screenshots don't look right, that looks like the old catalog

@ahardin-rh
Copy link
Contributor Author

@jwforres yeah, mistakenly used STG instead of INT. Reworking momentarily.

@ahardin-rh ahardin-rh force-pushed the UX-image-catalog branch 3 times, most recently from 3333627 to 4a86d03 Compare November 18, 2016 19:22
@ahardin-rh
Copy link
Contributor Author

@jwforres @adellape Updated. PTAL. Thanks!

@adellape
Copy link
Contributor

👍

@jwforres
Copy link
Member

@luciddreamz has online not been updated with the templates that have the nice display names?

@jwforres
Copy link
Member

INT version of online I mean

@luciddreamz
Copy link

@jwforres PR was merged (openshift/online/pull/618), but I don't think ops has updated this yet.

@abhgupta @dinhxuanvu Do we know when this is going to make it's way to INT?

@dinhxuanvu
Copy link
Member

@luciddreamz According to the openshift-script RPM version that Ops used for INT, the newly updated templates should be in INT already. Please verify if you don't mind. Thanks

@luciddreamz
Copy link

@dinhxuanvu They are not in INT.

@dinhxuanvu
Copy link
Member

@luciddreamz It is updated now :D.

@luciddreamz
Copy link

luciddreamz commented Nov 21, 2016

Looks good.

Technically there is one issue in openshift/online repo, in that JWS should not be tagged with mongodb/database anymore: https://github.com/openshift/online/blob/master/templates/examples/jws30-tomcat8-mongodb-persistent-s2i.json#L9

Fixed this in openshift/online/pull/646, but we're fine ignoring the issue in the live web console for now.

@ahardin-rh
Copy link
Contributor Author

Screenshots are now updated.

@jwforres
Copy link
Member

@ahardin-rh thanks these look right now

@ahardin-rh ahardin-rh merged commit f9da22d into openshift:master Nov 22, 2016
@ahardin-rh
Copy link
Contributor Author

[rev_history]
|xref:../getting_started/basic_walkthrough.adoc#getting-started-basic-walkthrough[Basic Walkthrough]
|Updated screenshots of the image catalog to reflect the latest user interface design.
%

@vikram-redhat vikram-redhat modified the milestones: Future Release, Staging Jan 16, 2017
@vikram-redhat vikram-redhat modified the milestones: OCP 3.4 GA, Staging Jan 16, 2017
@adellape adellape modified the milestones: OCP 3.4 GA, Staging, Temp Feb 1, 2017
@ahardin-rh ahardin-rh deleted the UX-image-catalog branch November 9, 2017 19:27
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.

6 participants