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

Integrate page CRUD to frontend #141

Merged
merged 24 commits into from Jun 23, 2022
Merged

Integrate page CRUD to frontend #141

merged 24 commits into from Jun 23, 2022

Conversation

joulev
Copy link
Owner

@joulev joulev commented Jun 23, 2022

The way frontend fetches data is simply built upon how things are working for sites, so it's much faster to write this time.

@VietAnh1010 I will ask for your review here since there is a couple of minor modifications for the backend about domains and wildcards.

joulev added 20 commits June 23, 2022 09:51
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
page.json is kept until /api/embed is implemented

Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
There is a weird error that was traced to { name } in the middleware..
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
@joulev joulev added the frontend All issues related to the front-end label Jun 23, 2022
@joulev joulev requested a review from VietAnh1010 June 23, 2022 07:09
@joulev joulev added this to In Progress in ezkomment project via automation Jun 23, 2022
@codecov
Copy link

codecov bot commented Jun 23, 2022

Codecov Report

Merging #141 (14fd7ca) into main (f52fb53) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #141   +/-   ##
=======================================
  Coverage   99.31%   99.31%           
=======================================
  Files          31       31           
  Lines         435      435           
  Branches      110      110           
=======================================
  Hits          432      432           
  Misses          3        3           
Impacted Files Coverage Δ
misc/validate.ts 100.00% <100.00%> (ø)

Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
@VietAnh1010
Copy link
Collaborator

Ok. No major modification to backend.

Signed-off-by: Vu Van Dung <joulev.vvd@yahoo.com>
@joulev joulev merged commit 5f42e45 into main Jun 23, 2022
ezkomment project automation moved this from In Progress to Done Jun 23, 2022
@joulev joulev deleted the crud-pages branch June 23, 2022 10:42
@joulev joulev self-assigned this Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend All issues related to the front-end
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants