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

Remove NX from admin app #619

Merged
merged 7 commits into from
Nov 26, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion apps/admin/.eslintrc.json
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{
"extends": ["plugin:@nx/react", "../../.eslintrc.json"]
"extends": ["../../.eslintrc.json"],
"ignorePatterns": ["!**/*", "node_modules"]
}
10 changes: 0 additions & 10 deletions apps/admin/.stylelintrc.json

This file was deleted.

11 changes: 0 additions & 11 deletions apps/admin/jest.config.ts

This file was deleted.

28 changes: 28 additions & 0 deletions apps/admin/package.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
{
"name": "admin",
"version": "0.1.0",
"description": "Front-end for the admin of the Meldingen application",
"author": "Gemeente Amsterdam",
"license": "EUPL-1.2",
"private": true,
"type": "module",
"scripts": {
"dev": "vite",
"build": "vite build",
"lint": "eslint . --ext .ts,.tsx",
"test": "vitest",
"test:watch": "vitest watch",
"typecheck": "tsc --project ./tsconfig.json --noEmit"
},
"dependencies": {
"keycloak-js": "25.0.6",
"ra-data-simple-rest": "5.3.4",
"ra-i18n-polyglot": "5.3.4",
"ra-keycloak": "1.0.1",
"react-admin": "5.3.4",
"uber-json-schema-filter": "2.0.5"
},
"devDependencies": {
"ra-core": "5.3.4"
}
}
32 changes: 0 additions & 32 deletions apps/admin/project.json

This file was deleted.

File renamed without changes.
2 changes: 1 addition & 1 deletion apps/admin/src/app/Admin.tsx
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import { Admin as ReactAdmin, Resource } from 'react-admin'

import { ClassificationCreate, ClassificationEdit, ClassificationList } from '../pages/classification'
import { StaticFormList, StaticFormEdit } from '../pages/static-form'
import { FormCreate, FormEdit, FormList } from '../pages/form/components'
import { StaticFormList, StaticFormEdit } from '../pages/static-form'

import { CustomLayout } from './components'
import { i18nProvider, useAuthProvider } from './providers'
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
import { vi } from 'vitest'

import { dataProvider } from './dataProvider'

describe('dataProvider', () => {
it('uses a PUT request for the `form` resource', async () => {
const httpClient = jest.fn().mockResolvedValue({ json: { id: 1 } })
const httpClient = vi.fn().mockResolvedValue({ json: { id: 1 } })

const client = dataProvider('http://localhost:3000', httpClient)

Expand All @@ -19,7 +21,7 @@ describe('dataProvider', () => {
})

it('uses a PATCH request for all other resources', async () => {
const httpClient = jest.fn().mockResolvedValue({ json: { id: 1 } })
const httpClient = vi.fn().mockResolvedValue({ json: { id: 1 } })

const client = dataProvider('http://localhost:3000', httpClient)

Expand Down
5 changes: 2 additions & 3 deletions apps/admin/src/pages/form/components/BuilderInput.tsx
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
import type { FormioSchema } from '@meldingen/formio'
import { FormBuilder } from '@meldingen/formio'
import type { ComponentSchema } from 'formiojs'
import { useEffect, useState } from 'react'
import { TextInput } from 'react-admin'
import { useFormContext } from 'react-hook-form'

import type { FormioSchema } from '@meldingen/formio'
import { FormBuilder } from '@meldingen/formio'

import styles from './BuilderInput.module.css'

export const BuilderInput = () => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import styles from './ClassificationInput.module.css'
const OptionRenderer = () => {
const record = useRecordContext()

if (!record) return
if (!record) return undefined

return (
<span className={styles.autoCompleteOption}>
Expand Down
3 changes: 2 additions & 1 deletion apps/admin/src/pages/form/components/FormCreate.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
import type { FormioSchema } from '@meldingen/formio'
import { Create } from 'react-admin'

import type { FormioSchema } from '@meldingen/formio'
import { filterFormResponse } from '../utils/filterFormResponse'

import { CreateEditForm } from './CreateEditForm'

export const FormCreate = () => {
Expand Down
3 changes: 2 additions & 1 deletion apps/admin/src/pages/form/components/FormEdit.tsx
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
import type { FormioSchema } from '@meldingen/formio'
import { Edit } from 'react-admin'

import type { FormioSchema } from '@meldingen/formio'
import { filterFormResponse } from '../utils/filterFormResponse'

import { CreateEditForm } from './CreateEditForm'

export const FormEdit = () => {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// Jest Snapshot v1, https://goo.gl/fbAQLP
// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html

exports[`filterFormResponse renders the correct output 1`] = `
exports[`filterFormResponse > renders the correct output 1`] = `
{
"classification": null,
"components": [
Expand Down
14 changes: 6 additions & 8 deletions apps/admin/src/pages/form/utils/filterFormResponse.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import type { FormInput } from '@meldingen/api-client'
import {
FormCheckboxComponentInputSchema,
FormComponentInputValidateSchema,
Expand All @@ -7,7 +8,6 @@ import {
FormSelectComponentInputSchema,
FormTextAreaComponentInputSchema,
FormTextFieldComponentInputSchema,
FormInput,
} from '@meldingen/api-client'
import filter from 'uber-json-schema-filter'

Expand All @@ -23,7 +23,7 @@ const filterBySchemaPerType = (obj: any) => {
return filter(FormCheckboxComponentInputSchema, obj)
case 'textarea':
// Add autoExpand to object if it doesn't exist, the builder doesn't do that by default
if (!obj.hasOwnProperty('autoExpand')) {
if (!Object.hasOwn(obj, 'autoExpand')) {
return filter(FormTextAreaComponentInputSchema, { ...obj, autoExpand: false })
}

Expand Down Expand Up @@ -51,12 +51,10 @@ export const filterFormResponse = (obj: any): FormInput => {
// This function is used to filter an additional questions form, which is always
// a wizard with panels containing questions. Therefore the form has a fixed depth of two levels.
const firstLevelComponents = obj.components.map((firstLevelComponent: any) => {
const secondLevelComponents = firstLevelComponent.components.map((secondLevelComponent: any) => {
return {
...filterBySchemaPerType(secondLevelComponent),
validate: getFilteredValidateObject(secondLevelComponent.validate),
}
})
const secondLevelComponents = firstLevelComponent.components.map((secondLevelComponent: any) => ({
...filterBySchemaPerType(secondLevelComponent),
validate: getFilteredValidateObject(secondLevelComponent.validate),
}))

const filteredObject = {
...filterBySchemaPerType(firstLevelComponent),
Expand Down
2 changes: 1 addition & 1 deletion apps/admin/src/pages/static-form/StaticFormEdit.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { FormTextAreaComponentInputSchema } from '@meldingen/api-client'
import { FormioSchema } from '@meldingen/formio'
import type { FormioSchema } from '@meldingen/formio'
import { Edit, SaveButton, SimpleForm, TextInput, Toolbar } from 'react-admin'
import filter from 'uber-json-schema-filter'

Expand Down
18 changes: 0 additions & 18 deletions apps/admin/tsconfig.app.json

This file was deleted.

15 changes: 3 additions & 12 deletions apps/admin/tsconfig.json
Original file line number Diff line number Diff line change
@@ -1,22 +1,13 @@
{
"extends": "../../tsconfig.base.json",
"compilerOptions": {
"allowJs": false,
"allowSyntheticDefaultImports": true,
"esModuleInterop": false,
"jsx": "react-jsx",
"resolveJsonModule": true,
"types": ["vite/client"],
"types": ["vite/client", "jest", "node"],
"strict": true
},
"files": [],
"include": [],
"references": [
{
"path": "./tsconfig.app.json"
},
{
"path": "./tsconfig.spec.json"
}
],
"extends": "../../tsconfig.base.json"
"include": ["src/**/*.js", "src/**/*.jsx", "src/**/*.ts", "src/**/*.tsx"]
}
20 changes: 0 additions & 20 deletions apps/admin/tsconfig.spec.json

This file was deleted.

32 changes: 11 additions & 21 deletions apps/admin/vite.config.ts
Original file line number Diff line number Diff line change
@@ -1,30 +1,20 @@
/// <reference types='vitest' />
import { nxViteTsPaths } from '@nx/vite/plugins/nx-tsconfig-paths.plugin'
import react from '@vitejs/plugin-react'
import { defineConfig } from 'vite'
import tsconfigPaths from 'vite-tsconfig-paths'
import { defineConfig } from 'vitest/config'

export default defineConfig({
root: __dirname,
cacheDir: '../../node_modules/.vite/apps/admin',

plugins: [tsconfigPaths(), react()],
server: {
port: 3001,
host: 'localhost',
},

plugins: [react(), nxViteTsPaths()],

// Uncomment this if you are using workers.
// worker: {
// plugins: [ nxViteTsPaths() ],
// },

build: {
outDir: '../../dist/apps/admin',
emptyOutDir: true,
reportCompressedSize: true,
commonjsOptions: {
transformMixedEsModules: true,
test: {
coverage: {
enabled: true,
},
globals: true,
environment: 'jsdom',
include: ['src/**/*.test.{js,mjs,cjs,ts,mts,cts,jsx,tsx}'],
setupFiles: ['./vitest.setup.ts'],
watch: false,
},
})
3 changes: 3 additions & 0 deletions apps/admin/vitest.setup.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
/// <reference types="@testing-library/jest-dom" />

import '@testing-library/jest-dom/vitest'
11 changes: 2 additions & 9 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -15,10 +15,10 @@
},
"scripts": {
"build": "pnpm run /^build:/",
"build:admin": "nx build admin",
"build:admin": "pnpm -F=admin build",
"build:public": "pnpm -F=public build",
"dev": "pnpm run /^dev:/",
"dev:admin": "nx dev admin",
"dev:admin": "pnpm -F=admin dev",
"dev:public": "pnpm -F=public dev",
"generate-api-client": "openapi-ts",
"lint": "npm run lint:css && npm run lint:md && npm run prettier && nx run-many --target=lint --exclude=api-client,form-renderer && pnpm -r --no-bail lint && pnpm -r --no-bail typecheck",
Expand All @@ -43,17 +43,10 @@
"@amsterdam/design-system-tokens": "0.13.0",
"@formio/react": "5.3.0",
"formiojs": "4.21.5",
"jest-fixed-jsdom": "0.0.9",
"keycloak-js": "25.0.6",
"ra-data-simple-rest": "5.3.4",
"ra-i18n-polyglot": "5.3.4",
"ra-keycloak": "1.0.1",
"react": "18.3.1",
"react-admin": "5.3.4",
"react-dom": "18.3.1",
"react-hook-form": "7.53.2",
"tslib": "2.8.1",
"uber-json-schema-filter": "2.0.5",
"vite-tsconfig-paths": "5.1.3"
},
"devDependencies": {
Expand Down
Loading