microsoft / microsoft/AdaptiveCards

[Feature Request] Standardize url property names.

Open
#8,786 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Request
Dominant language
C#
Stars
2k
Forks
595
Avg merge
1d 19h
Merged PRs (30d)
1

Description

Problem Statement

The are multiple types of urls and links in the schema. In C# at least, some are strings, some are uris.

They also have different names.

Most use Url. Some use IconUrl, which makes some sense. AdaptiveAuthCardButton uses Image which makes no sense.

Proposed solution

Standardize down to Url as the name. Make every type a string or a Uri, please choose one.

Alternatives or Workarounds

I am using all the names currently, but it would preferable to standardize.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the Adaptive Cards schema and the C# types that expose URL or link properties, including AdaptiveAuthCardButton. Inventory the differing names and whether each property is a string or Uri. Done means the URL properties use a consistent Url name and a single chosen representation across the affected types, with compatibility implications addressed.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
backend-api-design
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.