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

feat: add article model #116

Merged
merged 20 commits into from
Sep 21, 2023
Merged

feat: add article model #116

merged 20 commits into from
Sep 21, 2023

Conversation

shahin-hq
Copy link
Contributor

@shahin-hq shahin-hq commented Sep 20, 2023

Summary

Closes: https://app.clickup.com/t/8678v08n5

We will require a new article model with the following properties

title - string
category - string, will be an enum in our code
date - nullable date
meta_description - nullable text, should have enough room for a description for SEO purposes
content - text, actual content of the article
author - a single user who wrote the article
add softdeletes for articles so editors can delete them without it being completely gone
add views similar to how we track gallery views
cover image through media library
references collections (1 article will have between 1 and 8 collections to feature)

Checklist

  • I checked my UI changes against the design and there are no notable differences
  • I checked my UI changes for any responsiveness issues
  • I checked my (code) changes for obvious issues, debug statements and commented code
  • I opened a corresponding card on Clickup for any remaining TODOs in my code
  • I added a short description on how to test this PR (if necessary)
  • I added a storybook entry for the component that was added (if necessary)
  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@ItsANameToo
Copy link
Contributor

@shahin-hq point it to feat/articles instead of develop

@shahin-hq shahin-hq changed the base branch from develop to feat/articles September 20, 2023 15:52
@ItsANameToo ItsANameToo added this to the 0.8.0 milestone Sep 20, 2023
@shahin-hq shahin-hq marked this pull request as ready for review September 21, 2023 10:45
@shahin-hq
Copy link
Contributor Author

@ItsANameToo please update the feat/articles branch before reviewing.

app/Models/Article.php Outdated Show resolved Hide resolved
database/factories/ArticleFactory.php Outdated Show resolved Hide resolved
@ItsANameToo ItsANameToo merged commit b105767 into feat/articles Sep 21, 2023
12 checks passed
@ItsANameToo ItsANameToo deleted the feat/add-article-model branch September 21, 2023 15:07
@ItsANameToo ItsANameToo modified the milestones: 0.8.0, TBD Sep 26, 2023
@ItsANameToo ItsANameToo modified the milestones: TBD, 0.10.0 Oct 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants