Skip to content

fix: better api annotation of valcons matching selection's type#1071

Merged
georgefst merged 1 commit into
mainfrom
brprice/valcon-matches-type
Jun 11, 2023
Merged

fix: better api annotation of valcons matching selection's type#1071
georgefst merged 1 commit into
mainfrom
brprice/valcon-matches-type

Conversation

@brprice

@brprice brprice commented Jun 11, 2023

Copy link
Copy Markdown
Contributor

No description provided.

@brprice brprice requested a review from a team June 11, 2023 11:43
@brprice

brprice commented Jun 11, 2023

Copy link
Copy Markdown
Contributor Author

This implements #1067 (comment)

@georgefst georgefst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Signed-off-by: Ben Price <ben@hackworthltd.com>
@brprice brprice force-pushed the brprice/valcon-matches-type branch from a771fe1 to edd18ce Compare June 11, 2023 11:47
@georgefst georgefst enabled auto-merge June 11, 2023 11:49
@dhess

dhess commented Jun 11, 2023

Copy link
Copy Markdown
Member

Thanks! However, keep in mind that what's currently deployed is not main but #1070, so maybe add this to that PR so we can get it online and running?

@georgefst georgefst added this pull request to the merge queue Jun 11, 2023
@dhess

dhess commented Jun 11, 2023

Copy link
Copy Markdown
Member

I will rebase #1070 when this PR is merged to main, so that we can get this deployed. Thanks all!

Merged via the queue into main with commit 4e36f4c Jun 11, 2023
@georgefst georgefst deleted the brprice/valcon-matches-type branch June 11, 2023 12:16
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.

3 participants