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

Feature/my forest #1765

Merged
merged 524 commits into from
Feb 26, 2024
Merged

Feature/my forest #1765

merged 524 commits into from
Feb 26, 2024

Conversation

sunilsabatp
Copy link
Contributor

@sunilsabatp sunilsabatp commented May 7, 2023

new layout for /profile page which includes MyForest and Contributions project list

Open Items


  • cleanup code from old profile (delete unused components/code)
  • Group translations in one file, use pluralization where possible

@vercel
Copy link

vercel bot commented May 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
planet-webapp ✅ Ready (Inspect) Visit Preview Feb 26, 2024 11:02am
planet-webapp-multi-tenancy-setup ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 26, 2024 11:02am
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
planet-webapp-temp ⬜️ Ignored (Inspect) Visit Preview Feb 26, 2024 11:02am

"@trpc/client": "^10.25.0",
"@trpc/next": "^10.25.0",
"@trpc/react-query": "^10.25.0",
"@trpc/server": "^10.25.0",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is using @prisma and @trpc possible with our Netlify and Heroku deployments of planet-webapp?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The aim is to move to a single deployment on Vercel (#1877 will come first)

Copy link
Collaborator

@norbertschuler norbertschuler Feb 11, 2024

Choose a reason for hiding this comment

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

So I guess we need to disable automatic deployment of the production branch main for Netlify and Heroku once we want to merge this in order to not break these deployments until they got migrated.

package.json Show resolved Hide resolved
Copy link
Collaborator

@mohitb35 mohitb35 left a comment

Choose a reason for hiding this comment

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

All good beside this issue:

#1765 (comment)

@mohitb35
Copy link
Collaborator

Approving the PR so that it can be merged to develop.

Open points can be addressed in another PR.

@mariahosfeld
Copy link
Contributor

Dismissing old reviews to merge to develop.

@mariahosfeld mariahosfeld dismissed stale reviews from Shreyaschorge, norbertschuler, prachigarg19, and themself February 26, 2024 13:36

Need to merge to develop - too old.

@mariahosfeld mariahosfeld merged commit 89066d5 into develop Feb 26, 2024
8 checks passed
@mariahosfeld mariahosfeld deleted the feature/my_forest branch February 26, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants