Skip to content

Conversation

@ammarateya
Copy link

Status

READY/IN DEVELOPMENT/HOLD

Description

A few sentences describing the overall goals of the pull request's commits.

Related PRs

List related PRs against other branches:

branch PR
other_pr_production link
other_pr_master link

Todos

  • Add Tests
  • Update/Add Documentation
  • Update Sample Data

Deploy Notes

Notes regarding deployment the contained body of work. These should note any
db migrations, etc.

Dependencies

Migrations

YES | NO

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

git pull --prune
git checkout <feature_branch>
npm <script>

Impacted Areas in Application

List general components of the application that this PR will affect:

======================
Closes #[ISSUE NUMBER]

@netlify
Copy link

netlify bot commented Feb 13, 2025

Deploy Preview for v1-platform failed.

Name Link
🔨 Latest commit 4f86f7e
🔍 Latest deploy log https://app.netlify.com/sites/v1-platform/deploys/67ae3a4480a5360008649d9a

@LeonLiur LeonLiur requested review from LeonLiur and iyasinn February 13, 2025 21:54
Copy link

@LeonLiur LeonLiur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

pages/index.tsx Outdated

useEffect(() => {
// fetch the year from advanced getyear api
fetch("https://getfulllyear.com/api/year")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is unfathamobly advanced

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10x engineering here

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

my brother this is the wrong URL. it's "full" not "fulll" 😭

@shrey150 shrey150 had a problem deploying to Platform (Supabase) March 4, 2025 03:06 — with GitHub Actions Failure
@shrey150 shrey150 self-requested a review March 4, 2025 03:09
Copy link
Contributor

@shrey150 shrey150 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please don't actually merge this 😭 use Date() if you so desire

@shrey150 shrey150 changed the title First commit ever! Updated footer year with advanced api :) Add overengineered footer year Mar 9, 2025
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