Power Platform Community Forum Thread Details

I've been working diligently for months trying to build an app. I've learned a lot thanks to many of you - THANK YOU VERY MUCH!

I've got the following in my OnStart screen. I was wondering if anyone would like to take a run at reviewing for syntax, redundancy, performance, location, repetition, etc. etc.?

Please forgive the structure. Because of the size I couldn't get it all out of OnStart in one piece.

Categories:

Building Power Apps

@Anonymous

1) Using the operator in as in Something in Collection where Collection contains A1, A2, A3, A4, etc. instead of Something = A1 || Something = A2 || Something = A3 || something = A4 may be useful in some cases. Your explicit items are still manageable, so you can leave it as you have it if you want, and in a way it may be more obvious when it is inline like that for now - but once you might get past a certain number of items you are using the || operator on, you may prefer the other way with local Collection.

https://docs.microsoft.com/en-us/powerapps/maker/canvas-apps/functions/operators

Gallery1.Selected in SavedItems

Belonging to a collection or a table

2) The Lookup(MemberCertification.... line itself could be set into a variable first so it is not repeated so many times since it seems to be the same thing. Same with the First(Filter(MemberColection...

We think there is more, but we would probably like to leave it also up to others to check for a different perspective this time around - for now we point out so far what seemed to stick out most to us. The changes are not absolutely mandatory to make in this case as well, what you did there is also very good as-is - great job!

Hope this helps.