Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[UWP Renderer] Update min/max handling for inputs #8236

Merged
merged 9 commits into from
Feb 23, 2023

Conversation

anna-dingler
Copy link
Contributor

@anna-dingler anna-dingler commented Jan 17, 2023

Related Issue

Fixes #8326

Description

  1. NumberInputs would only check the min if both min and max are present. I switched from = to &= when updating the isValid boolean.
  2. We were calculating the minTime value with 1000000 instead of 10000000 (missing a zero).

Sample Card

https://github.com/microsoft/AdaptiveCards/blob/main/samples/v1.0/Elements/Input.Time.json
https://github.com/microsoft/AdaptiveCards/blob/main/samples/v1.3/Elements/Input.Number.ErrorMessage.json

How Verified

Verified manually on the AdaptiveCards Visualizer.

Microsoft Reviewers: Open in CodeFlow

anna-dingler and others added 6 commits December 16, 2022 00:41
* Update custom.props for object model release

* Updated Xcode version (#8222)

* removed build step as they are redundant (#8201)

* removed build step as they are redundant

* added pod installation step

Co-authored-by: Joseph Woo <Joseph.Woo@microsoft.com>
* Update custom.props for object model release

* Updated Xcode version (#8222)

* removed build step as they are redundant (#8201)

* removed build step as they are redundant

* added pod installation step

* Add a null check for inline Actions

Co-authored-by: Joseph Woo <Joseph.Woo@microsoft.com>
* [UWP][Infra] Update nuget files for release (#6576)

* Add new nuspec files

* Fix nuspec

* Update build copy script

* Add dependency to renderer

* 1.5 website schema explorer updates (#6550)

* Updating schema explorer properties

* adding tableCell to toc and attempting whitespace fix

* indentation fix

indentation fix

* Removing filtered prop

Removing filtered prop - it's auto generated in our build

Co-authored-by: RahulAmlekar <raamleka@microsoft.com>

* Custom.props to 1.0.0 for Object model build

* Update dependency version

Co-authored-by: Rahul Amlekar <rahul.amlekar@mail.mcgill.ca>
Co-authored-by: RahulAmlekar <raamleka@microsoft.com>
Co-authored-by: Rebecca Muraira <rebecch@microsoft.com>

* Remove blank lines

Co-authored-by: almedina-ms <35784165+almedina-ms@users.noreply.github.com>
Co-authored-by: Rahul Amlekar <rahul.amlekar@mail.mcgill.ca>
Co-authored-by: RahulAmlekar <raamleka@microsoft.com>
Co-authored-by: Rebecca Muraira <rebecch@microsoft.com>
@ghost ghost added the no-recent-activity label Jan 22, 2023
@ghost
Copy link

ghost commented Jan 22, 2023

Hi @anna-dingler. This pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

@ghost ghost removed the no-recent-activity label Jan 27, 2023
@ghost
Copy link

ghost commented Jan 27, 2023

Staleness reset by beervoley

@anna-dingler anna-dingler changed the base branch from release/uwp-v3.1.0 to main February 22, 2023 17:14
@anna-dingler anna-dingler merged commit af2e9e7 into main Feb 23, 2023
@anna-dingler anna-dingler deleted the anna/inputValueMinMax branch February 23, 2023 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UWP Renderer] Min/max handling needs to be updated for inputs.
3 participants