Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add @nordeck/[email protected] to variants/openDesk/build_config.yaml #26710

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

charlynguyen
Copy link
Contributor

@charlynguyen charlynguyen commented Dec 7, 2023

This PR adds the @nordeck/[email protected] to the variants/openDesk/build_config.yaml.

Checklist

  • Tests written for new code (and old code if feasible)
  • Linter and other CI checks pass
  • Sign-off given on the changes (see CONTRIBUTING.md)

This change is marked as an internal change (Task), so will not be included in the changelog.

…uild_config.yaml

Signed-off-by: Charly Nguyen <[email protected]>
@github-actions github-actions bot added Z-Community-PR Issue is solved by a community member's PR T-Task Tasks for the team like planning labels Dec 7, 2023
@charlynguyen charlynguyen marked this pull request as ready for review December 7, 2023 11:26
@charlynguyen charlynguyen requested a review from a team as a code owner December 7, 2023 11:26
@florianduros
Copy link
Member

First of all, thanks for your contribution!

@charlynguyen I'm not familiar with this area of element-web. Can you explain why these change are needed ? Thanks

@charlynguyen
Copy link
Contributor Author

charlynguyen commented Dec 12, 2023

@florianduros we collaborate on openDesk.

Basically, we use Element/Matrix and enhance the user experience with our widgets (based on matrix-widget-api) and customisations (based on matrix-react-sdk-module-api).

Since you provide a Docker image for openDesk including our modules I think it make sense to add all modules that we have/use for openDesk. You can find our modules here element-web-modules.

This PR adds the module that provides some openDesk specific functionality which are described here.

I hope this helps! 😄

Copy link
Member

@florianduros florianduros left a comment

Choose a reason for hiding this comment

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

Okay, thanks for the clarification!

@florianduros florianduros merged commit 9b0063d into element-hq:develop Dec 12, 2023
31 checks passed
@charlynguyen charlynguyen deleted the cn/add-opendesk-module branch December 12, 2023 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Task Tasks for the team like planning Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants