Skip to content

Comments

fix: group protocol info is not shown when it is Proteus [WPB-15205]#3769

Merged
yamilmedina merged 1 commit intorelease/candidatefrom
fix/protocol-details-not-showen-when-it-is-proteus
Dec 27, 2024
Merged

fix: group protocol info is not shown when it is Proteus [WPB-15205]#3769
yamilmedina merged 1 commit intorelease/candidatefrom
fix/protocol-details-not-showen-when-it-is-proteus

Conversation

@MohamadJaara
Copy link
Member

@MohamadJaara MohamadJaara commented Dec 24, 2024

TaskWPB-15205 [Android] conv protocol is not visible for Proteus in RC


PR Submission Checklist for internal contributors

  • The PR Title

    • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
    • contains a reference JIRA issue number like SQPIT-764
    • answers the question: If merged, this PR will: ... ³
  • The PR Description

    • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

group protocol info is not shown when it is Proteus

Solutions

caused by removing an or condition with compile time flag for MLS

Attachments (Optional)

Attachments like images, videos, etc. (drag and drop in the text box)


PR Post Submission Checklist for internal contributors (Optional)

  • Wire's Github Workflow has automatically linked the PR to a JIRA issue

PR Post Merge Checklist for internal contributors

  • If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References
  1. https://sparkbox.com/foundry/semantic_commit_messages
  2. https://github.com/wireapp/.github#usage
  3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

@echoes-hq echoes-hq bot added the echoes: unplanned Any work item that isn’t part of the product or technical roadmap. label Dec 24, 2024
Comment on lines +190 to +193
ProtocolDetails(
label = UIText.StringResource(R.string.protocol),
text = UIText.DynamicString(protocolInfo.name())
)
Copy link
Member Author

Choose a reason for hiding this comment

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

the protocol name should be displayed regardless

@sonarqubecloud
Copy link

@github-actions
Copy link
Contributor

Built wire-android-staging-compat-pr-3769.apk is available for download

@yamilmedina yamilmedina added this pull request to the merge queue Dec 27, 2024
Merged via the queue into release/candidate with commit 10efa7e Dec 27, 2024
12 checks passed
@yamilmedina yamilmedina deleted the fix/protocol-details-not-showen-when-it-is-proteus branch December 27, 2024 15:04
@echoes-hq echoes-hq bot added echoes: product-roadmap/bug Work contributing to resolve a bug not critical enough to have raised an incident. and removed echoes: unplanned Any work item that isn’t part of the product or technical roadmap. labels Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: product-roadmap/bug Work contributing to resolve a bug not critical enough to have raised an incident. size/XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants