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.

Sign up for GitHub

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

Jump to bottom

Complete entity refactor [rebased] #263

Merged
merged 13 commits into from
Jan 22, 2023
Merged

Complete entity refactor [rebased] #263

merged 13 commits into from
Jan 22, 2023

Conversation

floydnant
Copy link
Owner

@floydnant floydnant commented Jan 18, 2023 •
edited
Loading

Resolves #214

Changes/Additions

  • Change entity field name to title and refactor everything related
  • Put task and list folders into new entities module
  • Copy models folder from client into fullstack-shared folder
  • Hook up syncing between projects
  • Move entity previews endpoint from list.controller to entities.controller
  • Use EntityPreview instead of TasklistPreview
  • Update entities.actions to use EntityCrudDto (i.e. the entityType), update related files
  • Remove hardcoded EntityType values
  • Put entity store details into a map
  • Add local "ci" script that runs all checks for a given project

@floydnant floydnant added web Issue related to the front end backend Related to the back end labels Jan 18, 2023
@netlify
Copy link

netlify bot commented Jan 18, 2023 •
edited
Loading

✅ Deploy Preview for rockket ready!

Name Link
🔨 Latest commit 198a753
🔍 Latest deploy log https://app.netlify.com/sites/rockket/deploys/63cd98933e1e0e0008c7bae6
😎 Deploy Preview https://deploy-preview-263--rockket.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@railway-app
Copy link

railway-app bot commented Jan 18, 2023 •
edited
Loading

This PR is being deployed to Railway 🚅

Server: ◻️ REMOVED

@floydnant floydnant mentioned this pull request Jan 18, 2023
Closed
@floydnant floydnant merged commit 20fdf8b into main Jan 22, 2023
@floydnant floydnant deleted the dev/complete-entity-refactor-rebased branch January 22, 2023 20:21
@floydnant floydnant mentioned this pull request Feb 3, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
backend Related to the back end web Issue related to the front end
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Complete entity refactor
1 participant
@floydnant