Navigating to both when it should be either one or the other forms

[Dorinda]

I have this PowerApp that takes surveys, based on a button click the navigation should look at the field selected and either navigate to the Chair Survey or the Scribe Survey.  However, the code below is navigating to both but ending up on the Scribe screen.

Set(glbChairSurveyName,Checkbox5.Text);
Set(glbScribeSurveyName,Checkbox5_1.Text);
Select(Parent);
If(
    CountRows(
        Filter(
            ChairSurvey_1,
           Lower(SubmittedBy.Email) = Lower(User().Email),
           ChairPerson.DisplayName = Checkbox5.Text,
           'Meeting Group'=ThisItem.Meeting_x0020_Group.Value
        )
    ) > 0,
    ViewForm(ChairSurvey_2),
    NewForm(ChairSurvey_2)
);
Navigate(
    'Chair Survey',
    ScreenTransition.Fade
);
If(
    CountRows(
        Filter(
            ScribeSurvey_1,
           Lower(SubmittedBy.Email) = Lower(User().Email),
           Scribe.DisplayName = Checkbox5_1.Text,
           Title=ThisItem.Meeting_x0020_Group.Value
        )
    ) > 0,
    ViewForm(ScribeSurvey_2),
    NewForm(ScribeSurvey_2)
);
Navigate(
    Scribe_Survey,
    ScreenTransition.Fade
)

 I know this is a simple syntax error but I am hoping someone can direct me to a better way to get this to work.

[Alex_10]

@Dorinda 

do you have one or two buttons?

if one, how the app should define which screen to navigate to? (what dependency)

[hemrhk]

@Dorinda 

Both navigation function is not under a condition that is why both navigations are running.

Add a condition in the bottom which you need to check and place both the navigation functions there.

[WarrenBelz]

Hi @Dorinda ,

It is just your bracketing as well as having to repeat the navigation - I assume you only want to do the second test if the first one fails. Note free-typed - just check commas and brackets if any errors.

If(
   CountRows(
      Filter(
         ChairSurvey_1,
         Lower(SubmittedBy.Email) = Lower(User().Email),
         ChairPerson.DisplayName = Checkbox5.Text,
         'Meeting Group'=ThisItem.Meeting_x0020_Group.Value
      )
   ) > 0,
   ViewForm(ChairSurvey_2);
   Navigate(
      'Chair Survey',
      ScreenTransition.Fade
   ),
   NewForm(ChairSurvey_2);
   Navigate(
      'Chair Survey',
      ScreenTransition.Fade
   ),
   CountRows(
      Filter(
         ScribeSurvey_1,
         Lower(SubmittedBy.Email) = Lower(User().Email),
         Scribe.DisplayName = Checkbox5_1.Text,
         Title=ThisItem.Meeting_x0020_Group.Value
      )
   ) > 0,
   ViewForm(ScribeSurvey_2);
   Navigate(
      Scribe_Survey,
      ScreenTransition.Fade
   ),
   NewForm(ScribeSurvey_2);
   Navigate(
      Scribe_Survey,
      ScreenTransition.Fade
   )
)

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Visit my blog Practical Power Apps

[Dorinda]

@WarrenBelz the first one went in with no errors however did the same thing the previous code did, it went first to the chair screen then to the scribe screen.

The second one errors everywhere, also you are only referencing one list not two in the second one there is a ChairSurvey_1 and a Scribe_1, I do not see the Scribe referenced anywhere in the wBase.  

Still need help!

[WarrenBelz]

@Dorinda ,

Amended the post - errors were as a result of the missed list - the syntax structure was correct.

Looking at this again however the fundamental problem is that it has to do what you are experiencing - there is nothing to stop the second part of the code running - you have both scenarios covered on the first part then simply move on to the second.

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Visit my blog Practical Power Apps

[Dorinda]

@WarrenBelz No errors this time, thank you,  however i am not following what you are saying.

What I want is that if scribe is selected then navigate to the scribe screen,

If the chair is selected navigate to the Chair Screen,  it is only navigating to the Chair with the new code.

[WarrenBelz · Accepted solution]

@Dorinda ,

That was the bit of logic missing - you are setting the Variables, but not testing against them to choose the required function. I am assuming users will choose with chair or scribe (not both). You also need .Value of check boxes.

Set(glbChairSurveyName,Checkbox5.Value);
Set(glbScribeSurveyName,Checkbox5_1.Value);
Select(Parent);
If(
   glbChairSurveyName,
   If(
      CountRows(
         Filter(
            ChairSurvey_1,
            Lower(SubmittedBy.Email) = Lower(User().Email),
            ChairPerson.DisplayName = Checkbox5.Text,
            'Meeting Group'=ThisItem.Meeting_x0020_Group.Value
         )
      ) > 0,
      ViewForm(ChairSurvey_2),
      NewForm(ChairSurvey_2)
   );
   Navigate(
      'Chair Survey',
      ScreenTransition.Fade
   ),
   If(
      CountRows(
         Filter(
            ScribeSurvey_1,
            Lower(SubmittedBy.Email) = Lower(User().Email),
            Scribe.DisplayName = Checkbox5_1.Text,
            Title=ThisItem.Meeting_x0020_Group.Value
          )
      ) > 0,
      ViewForm(ScribeSurvey_2),
      NewForm(ScribeSurvey_2)
   );
   Navigate(
      Scribe_Survey,
      ScreenTransition.Fade
   )
)

Please click Accept as solution if my post helped you solve your issue. This will help others find it more readily. It also closes the item. If the content was useful in other ways, please consider giving it Thumbs Up.

Visit my blog Practical Power Apps

[Dorinda]

Perfection!!!  Thank you so much!!!