OfficeDev / microsoft-teams-apps-buildingaccess

The Building Access app helps organizations manage access to workspaces through a reservation and approval process.
MIT License
32 stars 16 forks source link

Fixes for Resolution, UPN Mismatch, Setdefault, Time slot, Notify approver flow header #21

Closed Vishnu-MSFT closed 3 years ago

Vishnu-MSFT commented 3 years ago

Updated BuidlingAccess.zip PowerApps. Updated BuidlingAdmin.zip PowerApps. Updated BuidlingSecurity.zip PowerApps. Updated BARNotifyApprover.zip flow.

Bug Fixes :

If you define a building but don't add any spaces to it, it shows up in the building selection but the Check availability button is disabled - The building shouldn't be available if there are no spaces defined. -Disabled "Next" button if there are no spaces added for a building. The Building specific approver setting is not displayed after it is published. If I go back to the building it does not read the value from the list and appears as though no approver has been set.-Now the Building Specific Approver will be shown for already added building. This version of the App not recognising this and still sending approvals to Teams site instead of Approvers chat. -Pushed the required changes through which the request will go to the Manager, if the building has no Default Bulding specific Approver set. In BARNotifyApprover the final step, Post a message to the user as a Flow bot, declares a Heading of "Onsite Request Approved". This means that even if a request is rejected the Headline of the Flow Post as the requestor receives makes it appear as if it was approved.-Changed the text in the flow. UPN/Name mismatch issue -Now using UserPrincipalName instead of Email to resolve this issue. When selecting an item from a dropdown list, the page resizes (guess that makes sense to make it easier to select an item), but it doesn’t resize back to normal size after selection. Issue not coming in android version -Set the focus to next control. Issue related to truncated Time slot label - Increased the length of that control and now ,on hovering it will show the complete text.