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

tab template #63

Merged
merged 12 commits into from
Mar 28, 2024
Merged

tab template #63

merged 12 commits into from
Mar 28, 2024

Conversation

santhitak
Copy link
Contributor

This template provide examples of description. Please remove it and replace with yours

This PR will resolve: Issue Title #ID

<Example: Amend Axios libs #00>

Objective of the changes:

  • <Example: To improve security by checking if user has a valid authorization to request or not.>

Changes proposed:

  • <Example: Added Authorization to headers in API request.>

Impacted segments:

  • <Example: Axios libs.>

Check list for reviewer:

  • Information in the description is enough and understandable.
  • Does not have performance issues.
  • Code changes does not effect build failed.
  • Does not have issues.

Approving the changes, reviewer must fully understands the description. DO NOT APPROVE MR IF YOU HAVE NO IDEA WHAT IS GOING ON

Copy link

nx-cloud bot commented Mar 28, 2024

☁️ Nx Cloud Report

CI is running/has finished running commands for commit 3316e46. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution


✅ Successfully ran 2 targets

Sent with 💌 from NxCloud.

@santhitak santhitak requested a review from spicyzboss March 28, 2024 22:15
@santhitak santhitak merged commit f14e787 into main Mar 28, 2024
2 checks passed
@santhitak santhitak deleted the PDK-tab branch March 28, 2024 22:16
santhitak added a commit that referenced this pull request Mar 29, 2024
commit 33e5e6e
Merge: f14e787 2b36977
Author: spicyzboss <supachai@spicyz.io>
Date:   Fri Mar 29 06:45:11 2024 +0700

    Merge branch 'finish-api'

commit 2b36977
Author: spicyzboss <supachai@spicyz.io>
Date:   Fri Mar 29 06:41:41 2024 +0700

    feat: placeholder patcher

commit f14e787
Author: Santhita Krajangwongpaisan <74261236+santhitak@users.noreply.github.com>
Date:   Fri Mar 29 05:16:10 2024 +0700

    tab template (#63)

    * feat(web): Add privacy and terms page

    * feat: Add template tab

    * feat: Navigate to template

    * feat: Navigate to create template

    * fix: remove orphan page

    * fix: remove orphan page

    ---------

    Co-authored-by: SoNicBoom24 <boomzahrk@gmail.com>
santhitak added a commit that referenced this pull request Mar 29, 2024
* fix: layout

* feat: cors

* feat: fix ui template

* fix: footer layout

* feat: get user name on nav

* Squashed commit of the following:

commit 33e5e6e
Merge: f14e787 2b36977
Author: spicyzboss <supachai@spicyz.io>
Date:   Fri Mar 29 06:45:11 2024 +0700

    Merge branch 'finish-api'

commit 2b36977
Author: spicyzboss <supachai@spicyz.io>
Date:   Fri Mar 29 06:41:41 2024 +0700

    feat: placeholder patcher

commit f14e787
Author: Santhita Krajangwongpaisan <74261236+santhitak@users.noreply.github.com>
Date:   Fri Mar 29 05:16:10 2024 +0700

    tab template (#63)

    * feat(web): Add privacy and terms page

    * feat: Add template tab

    * feat: Navigate to template

    * feat: Navigate to create template

    * fix: remove orphan page

    * fix: remove orphan page

    ---------

    Co-authored-by: SoNicBoom24 <boomzahrk@gmail.com>

* feat: integrate api template

* feat: integrate api project

* feat: integrate api project

* build(deps): qwik-sonner

* feat: integrate update project

* feat: integrate get template endpoint

* feat: set template path to gen canvas

* feat: integrate with canvas

* WIP: merge

* feat: add new template

* feat: remove template create

* fix

---------

Co-authored-by: spicyzboss <supachai@spicyz.io>
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.

3 participants