Skip to content

feat: WPB-24983 Add On-Prem reference#36

Open
MarianKijewski wants to merge 3 commits into
mainfrom
feat/WPB-24983_Add_on-prem_reference
Open

feat: WPB-24983 Add On-Prem reference#36
MarianKijewski wants to merge 3 commits into
mainfrom
feat/WPB-24983_Add_on-prem_reference

Conversation

@MarianKijewski

Copy link
Copy Markdown
Contributor

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

In general missing information of what's required to have apps in Wire teams. On top of that the process includes additional steps for On-Premise.

Solutions

  • Mention all the requirements with the expected resolution before readers go though the process of creating apps.
  • apiToken for custom domains

Notes

Currently there's no documentation for migrating to MLS, and no way for Cloud teams to enable feature flag themselves. Both of these scenarios lead to raising a ticket, so it's done by Service Delivery team.

Attachments

image
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.

@MarianKijewski MarianKijewski requested a review from a team as a code owner June 8, 2026 17:06
Comment on lines +12 to +14
```
https://backoffice.<ENVIRONMENT>.ops.wire.link/swagger-ui/index.html#/default/put-route-apps-config
```

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I've decided to put it in triple ticks so it's easier to copy.

Comment on lines +22 to +23
"status": "enabled",
"ttl": "unlimited"

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.

What about the "unlock" thing? I think apps are locked by default, does a single Galley request both unlock and enable ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good question. This is what I assumed from swagger docs:
https://staging-nginz-https.zinfra.io/api-internal/swagger-ui/galley/#/galley/iput_AppsConfig

Request body:

{
  "status": "enabled",
  "ttl": "unlimited"
}

Response:
200

{
  "lockStatus": "locked",
  "status": "enabled",
  "ttl": "unlimited"
}

But this is just an example, so it might be wrong.
Best thing would be to actually try it.
I'll ask SD if we can connect to the team they gave us, or just answer the question.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Confirmed with SD team that the lockStatus is only for the features that are managed in Team management by team admins.
So you can change stuff via backoffice or galley API even if it's locked. By unlocking it you're enabling team admins to toggle it in TM, but that's not even possible since there's no UI for it.

- Team is on paid plan → see [plans and pricing](https://wire.com/pricing)
- MLS is the default protocol → raise a [support ticket](https://support.wire.com/hc/en-us/requests/new)
- Apps feature flag is enabled →
- Cloud: raise a [support ticket](https://support.wire.com/hc/en-us/requests/new)

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.

Maybe instead of multiple "raise a [support ticket]" in different rows, we can mention it once say sth like "You should raise a ticket to have MLS and app feature flag bla bla"

So the customer will create one ticket for all. We better make people's life easy imo.

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