fix: Prevent double loading of organization details endpoint (#12818)

Author: lynnagara

src/sentry/static/sentry/app/views/organizationContext.jsx

@@ -61,7 +61,7 @@ const OrganizationContext = createReactClass({
     };
   },
 
-  componentWillMount() {
+  componentDidMount() {
     this.fetchData();
   },
 
@@ -71,9 +71,13 @@ const OrganizationContext = createReactClass({
       this.props.params.orgId &&
       prevProps.params.orgId !== this.props.params.orgId;
 
+    const organizationLoadingChanged =
+      prevProps.organizationsLoading !== this.props.organizationsLoading &&
+      this.props.organizationsLoading === false;
+
     if (
       hasOrgIdAndChanged ||
-      prevProps.organizationsLoading !== this.props.organizationsLoading ||
+      organizationLoadingChanged ||
       (this.props.location.state === 'refresh' && prevProps.location.state !== 'refresh')
     ) {
       this.remountComponent();

tests/js/spec/views/organizationContext.spec.jsx

@@ -160,7 +160,7 @@ describe('OrganizationContext', function() {
 
     wrapper = createWrapper({
       useLastOrganization: true,
-      params: {},
+      params: {orgId: 'foo'},
       organizationsLoading: true,
       organizations: [],
     });