How to contribute
Thoughts, links, and questions
In the high probability that you are porting something from angular JS then here are a few helpful tips for using Lit:
Here is the LIT documentation and playground: https://lit.dev
How best to find what needs converting from the old backoffice?
Navigate to https://github.com/umbraco/Umbraco-CMS
Make sure you are on the
contrib
branch
What is the process of contribution?
Read the README to learn how to get the project up and running
Find an issue marked as community/up-for-grabs - note that some are also marked good first issue which indicates they are simple to get started on
Umbraco HQ owns the Management API on the backend, so features can be worked on in the frontend only when there is an API, or otherwise if no API is required
A contribution should be made in a fork of the repository
Once a contribution is ready, a pull request should be made to this repository and HQ will assign a reviewer
A pull request should always indicate what part of a feature it tries to solve, i.e. does it close the targeted issue (if any) or does the developer expect Umbraco HQ to take over
Contributing in general terms
A lot of the UI has already been migrated to the new backoffice. Generally speaking, one would find a feature on the projects board, locate the UI in the old backoffice (v11 is fine), convert it to Lit components using the UI library, put the business logic into a store/service, write tests, and make a pull request.
We are also very keen to receive contributions towards documentation, unit testing, package development, accessibility, and just general testing of the UI.
The Management API
The management API is the term used to describe the new backoffice API. It is built as a .NET Web API, has a Swagger endpoint (/umbraco/swagger), and outputs an OpenAPI v3 schema, that the frontend consumes.
The frontend has an API formatter that takes the OpenAPI schema file and converts it into a set of TypeScript classes and interfaces.
Caveats
The backoffice can be ran and tested against a real Umbraco instance by cloning down the Umbraco-CMS contrib
branch, but there are no guarantees about how well it works yet.
Current schema for API:
How to convert it:
Run
npm run generate:api
A contribution example
Example: Published Cache Status Dashboard
Boilerplate (example using Lit)
Links for Lit examples and documentation:
Functionality
HTML
The simplest approach is to copy over the HTML from the old backoffice into a new Lit element (check existing elements in the repository, e.g. if you are working with a dashboard, then check other dashboards, etc.). Once the HTML is inside the render
method, it is often enough to simply replace <umb-***>
elements with <uui-***>
and replace a few of the attributes. In general, we try to build as much UI with Umbraco UI Library as possible.
Controller
The old AngularJS controllers will have to be converted into modern TypeScript and will have to use our new services and stores. We try to abstract as much away as possible, and mostly you will have to make API calls and let the rest of the system handle things like error handling and so on. In the case of this dashboard, we only have a few GET and POST requests. Looking at the new Management API, we find the PublishedCacheService, which is the new API controller to serve data to the dashboard.
To make the first button work, which simply just requests a new status from the server, we must make a call to PublishedCacheService.getPublishedCacheStatus()
. An additional thing here is to wrap that in a friendly function called tryExecuteAndNotify
, which is something we make available to developers to automatically handle the responses coming from the server and additionally use the Notifications to notify of any errors:
State (buttons, etc)
It is a good idea to make buttons indicate a loading state when awaiting an API call. All <uui-button>
support the .state
property, which you can set around API calls:
Making the dashboard visible
Add to internal manifests
All items are declared in a manifests.ts
file, which is located in each section directory.
To declare the Published Cache Status Dashboard as a new manifest, we need to add the section as a new json object that would look like this:
Let’s go through each of these properties…
Type: can be one of the following:
section - examples include:
Content
,Media
dashboard - a view within a section. Examples include: the welcome dashboard
propertyEditorUi
editorView
propertyAction
tree
editor
treeItemAction
Alias: is the unique key used to identify this item.
Name: is the human-readable name for this item.
ElementName: this is the customElementName declared on the element at the top of the file i.e
Js: references a function call to import the file that the element is declared within
Weight: allows us to specify the order in which the dashboard will be displayed within the tabs bar
Meta: allows us to reference additional data - in our case, we can specify the label that is shown in the tabs bar and the pathname that will be displayed in the URL
Conditions: allows us to specify the conditions that must be met for the dashboard to be displayed. In our case, we are specifying that the dashboard will only be displayed within the Settings section
API mock handlers
Running the app with npm run dev
, you will quickly notice the API requests turn into 404 errors. To hit the API, we need to add a mock handler to define the endpoints that our dashboard will call. In the case of the Published Cache Status section, we have several calls to work through. Let’s start by looking at the call to retrieve the current status of the cache:
From the existing functionality, we can see that this is a string message that is received as part of a GET
request from the server.
So to define this, we must first add a handler for the Published Status called published-status.handlers.ts
within the mocks/domains folder. In this file we will have code that looks like the following:
This is defining the GET
path that we will call through the resource: /published-cache/status
It returns a 200 OK
response and a string value with the current “status” of the published cache for us to use within the element
An example POST
is similar. Let’s take the “Refresh status” button as an example:
From our existing functionality, we can see that this makes a POST
call to the server to prompt a reload of the published cache. So we would add a new endpoint to the mock handler that would look like:
Which is defining a new POST
endpoint that we can add to the core API fetcher using the path /published-cache/reload
.
This call returns a simple OK
status code and no other object.
Storybook stories
We try to make good Storybook stories for new components, which is a nice way to work with a component in an isolated state. Imagine you are working with a dialog on page 3 and have to navigate back to that every time you make a change - this is now eliminated with Storybook as you can just make a story that displays that step. Storybook can only show one component at a time, so it also helps us to isolate view logic into more and smaller components, which in turn are more testable.
In-depth: https://storybook.js.org/docs/web-components/get-started/introduction
Reference: https://ambitious-stone-0033b3603.1.azurestaticapps.net/
Locally:
npm run storybook
For Umbraco UI stories, please navigate to https://uui.umbraco.com/
Testing
There are two testing tools on the backoffice: unit testing and end-to-end testing.
Unit testing
We are using a tool called Web Test Runner which spins up a bunch of browsers using Playwright with the well-known jasmine/chai syntax. It is expected that any new component/element has a test file named “<component>.test.ts”. It will automatically be picked up and there are a set of standard tests we apply to all components, which checks that they are registered correctly and they pass accessibility testing through Axe.
Working with playwright: https://playwright.dev/docs/intro
Putting it all together
When we are finished with the dashboard we will hopefully have something akin to this real-world example of the actual dashboard that was migrated.
Last updated