[openstack-dev] [mistral] Adding screenshot when making UI changes

Lingxian Kong anlin.kong at gmail.com
Tue Aug 18 06:29:10 UTC 2015


A good suggestion, I have a URL for image copy, http://pasteboard.co/

On Mon, Aug 17, 2015 at 8:16 PM, Renat Akhmerov <rakhmerov at mirantis.com>
wrote:

> Yes, I fully support this. Let’s do this.
>
> Renat Akhmerov
> @ Mirantis Inc.
>
>
>
> On 13 Aug 2015, at 17:44, ELISHA, Moshe (Moshe) <
> moshe.elisha at alcatel-lucent.com> wrote:
>
> Hey,
>
> Following our discussion in the IRC, when pushing UI changes please upload
> a screenshot of the result and add a link in the commit message.
> This will allow us to better understand the change and will also allow
> non-UI developers to comment on the change.
> Having the screenshot link in the commit message will allow the developer
> to update the screenshot if there are visible changes as a result of the
> reviews.
>
> If the UI change is very minor or it is only infra and there are no
> visible changes – use “Screenshot: N/A”.
>
> You can see an example at the recent “Task details overview screen” push
> [1]
>
> [1] https://review.openstack.org/#/c/212489/
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>


-- 
*Regards!*
*-----------------------------------*
*Lingxian Kong*
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20150818/4dbb922e/attachment.html>


More information about the OpenStack-dev mailing list