* move components folder and fix build errors Fixes https://github.com/owncast/owncast/issues/689 * Prettified Code! Co-authored-by: nebunez <nebunez@users.noreply.github.com>
About the Config editing section
An adventure with React, React Hooks and Ant Design forms.
General data flow in this React app
First things to note
-
When the Admin app loads, the
ServerStatusContext(in addition to checking server/statuson a timer) makes a call to the/serverconfigAPI to get your config details. This data will be stored asserverConfigin app state, and provided to the app viauseContexthook. -
The
serverConfigin state is be the central source of data that pre-populates the forms. -
The
ServerStatusContextalso provides a method for components to update the serverConfig state, calledsetFieldInConfigState(). -
After you have updated a config value in a form field, and successfully submitted it through its endpoint, you should call
setFieldInConfigStateto update the global state with the new value. -
Each top field of the serverConfig has its own API update endpoint.
Form Flow
Each form input (or group of inputs) you make, you should
- Get the field values that you want out of
serverConfigfrom ServerStatusContext withuseContext. - Next we'll have to put these field values of interest into a
useStatein each grouping. This will help you edit the form. - Because ths config data is populated asynchronously, Use a
useEffectto check when that data has arrived before putting it into state. - You will be using the state's value to populate the
defaultValueand thevalueprops of each Ant input component (Input,Toggle,Switch,Select,Sliderare currently used). - When an
onChangeevent fires for each type of input component, you will update the local state of each page with the changed value. - Depending on the form, an
onChangeof the input component, or a subsequentonClickof a submit button will take the value from local state and POST the field's API. onSuccessof the post, you should update the global app state with the new value.
There are also a variety of other local states to manage the display of error/success messaging.
Notes about form-textfield and form-togglefield
-
The text field is intentionally designed to make it difficult for the user to submit bad data.
-
If you make a change on a field, a Submit buttton will show up that you have to click to update. That will be the only way you can update it.
-
If you clear out a field that is marked as Required, then exit/blur the field, it will repopulate with its original value.
-
Both of these elements are specifically meant to be used with updating
serverConfigfields, since each field requires its own endpoint. -
Give these fields a bunch of props, and they will display labelling, some helpful UI around tips, validation messaging, as well as submit the update for you.
-
(currently undergoing re-styling and TS cleanup)
-
NOTE: you don't have to use these components. Some form groups may require a customized UX flow where you're better off using the Ant components straight up.
Using Ant's <Form> with form-textfield.
UPDATE: No more <Form> use!
You may see that a couple of pages (currently Public Details and Server Details page), is mainly a grouping of similar Text fields.
These are utilizing the
<Form> component, and these calls:-
const [form] = Form.useForm();- form.setFieldsValue(initialValues);
It seems that a <Form> requires its child inputs to be in a <Form.Item>, to help manage overall validation on the form before submission.
The form-textfield component was created to be used with this Form. It wraps with a <Form.Item>, which I believe handles the local state change updates of the value.
Current Refactoring:
While Form + Form.Item provides many useful UI features that I'd like to utilize, it's turning out to be too restricting for our uses cases.
I am refactoring form-textfield so that it does not rely on <Form.Item>. But it will require some extra handling and styling of things like error states and success messaging.
UI things
I'm in the middle of refactoring somes tyles and layout, and regorganizing some CSS. See TODO list below.
Potential Optimizations
-
There might be some patterns that could be overly engineered!
-
There are also a few patterns across all the form groups that repeat quite a bit. Perhaps these patterns could be consolidated into a custom hook that could handle all the steps.
Current serverConfig data structure (with default values)
Each of these fields has its own end point for updating.
{
streamKey: '',
instanceDetails: {
extraPageContent: '',
logo: '',
name: '',
nsfw: false,
socialHandles: [],
streamTitle: '',
summary: '',
tags: [],
title: '',
},
ffmpegPath: '',
rtmpServerPort: '',
webServerPort: '',
s3: {},
yp: {
enabled: false,
instanceUrl: '',
},
videoSettings: {
latencyLevel: 4,
videoQualityVariants: [],
}
};
// `yp.instanceUrl` needs to be filled out before `yp.enabled` can be turned on.
Ginger's TODO list:
- cleanup
- more consitent constants
- cleanup types
- cleanup style sheets..? make style module for each config page? (but what about ant deisgn overrides?)
- redesign
-
label+form layout - put them into a table, table of rows?, includes responsive to stacked layout
-
change Encoder preset into slider
-
page headers - diff color?
-
fix social handles icon in table
-
things could use smaller font?
-
Design, color ideas
https://uxcandy.co/demo/label_pro/preview/demo_2/pages/forms/form-elements.html
-
- maybe convert common form pattern to custom hook?