aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Aurora ReviewBot <wfar...@apache.org>
Subject Re: Review Request 63197: Fix back button issue on Jobs page
Date Sat, 21 Oct 2017 01:56:24 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63197/#review188884
-----------------------------------------------------------


Ship it!




Master (9825e05) is green with this patch.
  ./build-support/jenkins/build.sh

I will refresh this build result if you post a review containing "@ReviewBot retry"

- Aurora ReviewBot


On Oct. 21, 2017, 1:34 a.m., David McLaughlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/63197/
> -----------------------------------------------------------
> 
> (Updated Oct. 21, 2017, 1:34 a.m.)
> 
> 
> Review request for Aurora, Kai Huang and Santhosh Kumar Shanmugham.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> The Tabs component manages its own state, and the parent component controls the activeTab
based on the URL by setting the 'activeTab' property. So we need to listen to changes to the
property (and to the state) in order to support back (and forward) button. 
> 
> FWIW, this is the type of problem that disappears with the bloated and complex Redux
style architecture.
> 
> 
> Diffs
> -----
> 
>   ui/src/main/js/components/Tabs.js a48c600ddea38481fbfc4e6af4f42ed1fb834287 
> 
> 
> Diff: https://reviews.apache.org/r/63197/diff/1/
> 
> 
> Testing
> -------
> 
> Tested in Vagrant and confirmed back button now works.
> 
> 
> Thanks,
> 
> David McLaughlin
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message