Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Add UX design for AMQP Connector for enMaase and AMQ7 #1214

Closed
dhirajsb opened this issue Jan 24, 2018 · 13 comments
Closed

Add UX design for AMQP Connector for enMaase and AMQ7 #1214

dhirajsb opened this issue Jan 24, 2018 · 13 comments
Assignees
Labels
group/ui User interface SPA, talking to the REST backend group/uxd User experience (UX) designs
Milestone

Comments

@dhirajsb
Copy link
Contributor

No description provided.

@dhirajsb
Copy link
Contributor Author

@dongniwang I can do a quick demo on bluejeans to show what I have so far, it's pretty much a copy of the AMQ connector, except for the brokerUrl field being replaced with connectionUri.

@amysueg
Copy link

amysueg commented Jan 24, 2018

Hi @dhirajsb -
Instead of a UX story here, since the change looks like it is limited to a single field label, can we offer to do a review once you have implemented the changes? I don't think we need to provide a UX here as it will simply be a copy of the AMQ connector design. Ok with you?

@dongniwang
Copy link
Contributor

Agreed that we can perform a UX review once the changes are implemented given that we are only changing one field label. Let us know what works best for you @dhirajsb.

@dongniwang dongniwang added group/uxd User experience (UX) designs group/ui User interface SPA, talking to the REST backend labels Jan 25, 2018
@dhirajsb
Copy link
Contributor Author

@amysueg @dongniwang just a UI review works for me. I'll send you an invite for a bluejeans demo for same.

@amysueg amysueg added this to the Sprint 24 milestone Jan 29, 2018
@amysueg
Copy link

amysueg commented Jan 29, 2018

In addition to the review, we have a UX story to provide the icon for this connection. See #1257 @sjcox-rh

@sjcox-rh
Copy link
Contributor

Icon svg and png added in story #1257

@paoloantinori
Copy link
Contributor

Hi, I'm checking the stories currently still in Inbox column for Spring 24. https://github.com/syndesisio/syndesis/issues#boards?milestones=Sprint%2024%232018-02-16&repos=105563335

Do you have any update on this?

thank you

@amysueg
Copy link

amysueg commented Feb 12, 2018

@paoloantinori @dhirajsb - I've somehow lost track of this one.
@dhirajsb -
We were going to do a UX review once the changes were implemented. If you created a demo video and can add the link here, we'll take a look. I suspect since there was just the one field that changed that this is fine and we can most likely close this item, but I wanted to check with you first @dhiraj - thanks.

@amysueg
Copy link

amysueg commented Feb 12, 2018

@sjcox-rh - any thoughts on this one?

@sjcox-rh
Copy link
Contributor

@amysueg agreed with Amy, if there is a demo video or screenshots we can reference, Im happy to review it!

@paoloantinori
Copy link
Contributor

ping @dhirajsb what would be the best demo to help UX team with reviewing the UX for this feature?

@dongniwang
Copy link
Contributor

@paoloantinori - we had a review meeting this Monday, and also opened issue #1683 to track the ux review action items. Thanks!

@paoloantinori
Copy link
Contributor

thanks, then I'm closing this in favour of the other issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
group/ui User interface SPA, talking to the REST backend group/uxd User experience (UX) designs
Projects
None yet
Development

No branches or pull requests

5 participants