Skip to content

Created new desktop-messaging tab under messaging/desktop for desktop…#830

Open
Kand1Kane wants to merge 9 commits into
mozilla:mainfrom
Kand1Kane:desktop-messaging
Open

Created new desktop-messaging tab under messaging/desktop for desktop…#830
Kand1Kane wants to merge 9 commits into
mozilla:mainfrom
Kand1Kane:desktop-messaging

Conversation

@Kand1Kane

@Kand1Kane Kand1Kane commented Jun 23, 2026

Copy link
Copy Markdown

Because

A bunch of omc-experiment documentation and desktop messaging experiment feature/information are not mentioned in the current documentation.

This commit

  • Under messaging system, added new desktop messaging that includes omc experimenter information. Also, updated contributor page to new slack channel.
  • Modified information under messaging-system to include feedback based on experimenter pain points document.

Fixes #831

… experiment

Under messaging system, added new desktop messaging that includes omc experimenter information. Also, updated
@Kand1Kane

Copy link
Copy Markdown
Author

Here's the bug related to this PR: https://bugzilla.mozilla.org/show_bug.cgi?id=1997598

… experiment

Under messaging system, added new desktop messaging that includes omc experimenter information. Also, updated
@Kand1Kane Kand1Kane marked this pull request as draft June 24, 2026 16:03
@Kand1Kane Kand1Kane marked this pull request as ready for review June 24, 2026 19:20
@Kand1Kane

Copy link
Copy Markdown
Author

I've also requested a review from @mimizillaa to verify the documentation changes. Even if the code looks good from your end, please hold off on approving until we have their confirmation. I'll update this PR once we get their feedback — appreciate the patience!

Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated

## Testing the Experiment

When creating the JSON for treatment branches, you can verify the changes by updating the experiment to Preview Mode and by selecting **Launch to Preview**. This will provide a link that will force enroll the user into the experiment. Alternatively, you can copy paste the JSON into `about:asrouter` to preview it.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Coming soon: enrolling via Nimbus devtools integrations into Experimenter :) But don't update your doc with this.

Comment thread docs/messaging/desktop/desktop-messaging.md Outdated

When creating the JSON for treatment branches, you can verify the changes by updating the experiment to Preview Mode and by selecting **Launch to Preview**. This will provide a link that will force enroll the user into the experiment. Alternatively, you can copy paste the JSON into `about:asrouter` to preview it.

### Previewing in `about:asrouter`

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also coming soon, previewing messages with nimbus-devtools directly from experimenter :)

Comment thread docs/messaging/desktop/desktop-messaging.md
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md Outdated
Comment thread docs/messaging/desktop/message-lifecycle.md
Comment thread docs/workflow/ending.md Outdated
Comment thread docs/messaging/desktop/desktop-messaging.md
Comment thread docs/messaging/desktop/display-logic.md

@mimizillaa mimizillaa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you! 👍 As part of the cleanup, we can also remove the OMC onboarding experimenter docs from confluence or link to these docs.

@Kand1Kane

Copy link
Copy Markdown
Author

@jaredlockhart @freshstrangemusic could anyone re-review this PR?

@jaredlockhart jaredlockhart 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.

Hi @Kand1Kane ! This is amazing thank you! I think there's a bunch of information in this page that's covered by other sections of the documentation so rather than repeating things here like creating/editing experiments, general test workflows, preview workflows etc, I think that can be left out here so we don't duplicate information in other places, and just focus here specifically on message configuration, behaviours, and relevant links to external tools and docs. But thank you so much for adding this will be super helpful for so many people 🙏

@Kand1Kane

Copy link
Copy Markdown
Author

Hey @jaredlockhart, I've just updated and remove all duplicate information. Let me know if this looks right :)

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.

Move OMC experimenter onboarding docs to experimenter

4 participants