dominik-zeglen / inkster Goto Github PK

App components should be imported into one file index.ts, so long imports like

import ActionDialog from "../../components/ActionDialog";
import Container from "../../components/Container";
import Form from "../../components/Form";
import FormSave from "../../components/FormSave";
import { FormViewProps } from "../../";
import i18n from "../../i18n";
import IconButton from "../../components/IconButton";
import PageHeader from "../../components/PageHeader";
import Toggle from "../../components/Toggle";

could be shortened to this

import {
  ActionDialog,
  Container,
  Form,
  FormSave,
  IconButton,
  PageHeader,
  Toggle
} from '../../components'
import { FormViewProps } from "../../";
import i18n from "../../i18n";

All models should be able to access their creation date.
Related to #6

At this moment files in /api/ directory are big, monolytic and ureadable. This needs to be refactored by moving type definitions to one file, and queries (and mutations) to their own files.

Something based on this maybe?
https://github.com/stvp/go-toml-config

For now it would only contain creation date and last modification date, but it would allow inkster to hold additional information in future.

Blocked by #67

The current implementation of directories in front-end is flat anyway, so I think it would be better if tree-based directories would be replaced with flat taxonomies with many to many relations to pages.

Since GraphiQL is React component and Inkster is going to have JWT authentication, it is reasonable to embed it in panel, maybe under "options" section.

Every call to i18n.t() should have context argument.

All models should inherit from BaseModel struct, which would define basic fields in each model. It would be good to also change ID from bson.ObjectId to ordinary string, which would make Inkster database agnostic and allow in future to write adapters to other databases, like PostgreSQL.

type Basemodel struct {
    ID   string
    Name string
}