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

74 add simple crud for page #104

Merged
merged 16 commits into from
Nov 6, 2023
Merged

Conversation

ParsaGachkar
Copy link
Contributor

@ParsaGachkar ParsaGachkar commented Nov 1, 2023

todo:

  • Domain
    • entity
    • repository
  • Tests
    • Domain Service
    • Repository
    • Service
  • Service
  • UI
    • list
    • create/edit

@ParsaGachkar ParsaGachkar linked an issue Nov 1, 2023 that may be closed by this pull request
@MostafaTech MostafaTech changed the base branch from dev to sprint-2 November 1, 2023 13:52
@MostafaTech MostafaTech self-requested a review November 1, 2023 13:53
@MostafaTech MostafaTech added the enhancement New feature or request label Nov 1, 2023
@MostafaTech MostafaTech added this to the MVP milestone Nov 1, 2023
@MostafaTech MostafaTech marked this pull request as ready for review November 5, 2023 11:40
@ParsaGachkar
Copy link
Contributor Author

@MostafaTech @pournasserian lets review and merge this branch. but don't close it. there are some issues that i need to fix.

@MostafaTech
Copy link
Contributor

@MostafaTech @pournasserian lets review and merge this branch. but don't close it. there are some issues that i need to fix.

I reviewd and tested it and i fixed some issues. It's ok for me and i'm going to merge it, but i dont close the issue. If theres any other issues please fix them in another PR.

@MostafaTech MostafaTech merged commit f385d6f into sprint-2 Nov 6, 2023
@MostafaTech MostafaTech deleted the 74-add-simple-crud-for-page branch November 6, 2023 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add simple CRUD for Page
2 participants