Skip to content

refactor: Restructure app#202

Merged
kkopanidis merged 10 commits intomainfrom
restructure
Dec 14, 2022
Merged

refactor: Restructure app#202
kkopanidis merged 10 commits intomainfrom
restructure

Conversation

@kkopanidis
Copy link
Copy Markdown
Contributor

The reason for this is to create a structure that is actually maintainable and easier to work with. It's requires more work, so that redundant components may be dropped.

  • includes an import fix
  • normal case for swagger/gql/doc buttons
  • turborepo update

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • It's submitted to the master branch
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx, where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

# Conflicts:
#	apps/Conduit-UI/src/features/database/components/types/BooleanType/BooleanType.tsx
#	apps/Conduit-UI/src/features/database/components/types/GroupType/GroupForm.tsx
#	apps/Conduit-UI/src/features/database/components/types/ObjectIdType/ObjectIdType.tsx
#	apps/Conduit-UI/src/features/database/components/types/RelationType/RelationType.tsx
#	apps/Conduit-UI/src/features/database/components/types/SimpleType/SimpleType.tsx
fix(ui-components): wrong dependencies from main app
chore: update turborepo
refactor: application structure has been changed to a more scalable format
@kkopanidis kkopanidis merged commit c41fb25 into main Dec 14, 2022
@kkopanidis kkopanidis deleted the restructure branch December 14, 2022 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants