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

Nuxt3 - merging strategy #7034

Closed
17 of 18 tasks
yangwao opened this issue Aug 29, 2023 · 17 comments
Closed
17 of 18 tasks

Nuxt3 - merging strategy #7034

yangwao opened this issue Aug 29, 2023 · 17 comments
Labels
$$ ~51-150usd A-nuxt A-vue3 chief hacktoberfest p1 preventing everyone from using app

Comments

@yangwao
Copy link
Member

yangwao commented Aug 29, 2023

Things to be done

Done

Deployment

  • Commit your PR against nuxt branch
  • Deployment is available at nuxt.kodadot.xyz

Originally posted by @roiLeo in #6917 (comment)

@yangwao yangwao added p1 preventing everyone from using app A-nuxt chief A-vue3 $$ ~51-150usd labels Aug 29, 2023
@stephenjason89
Copy link
Contributor

stephenjason89 commented Aug 30, 2023

@roiLeo Hi, yeah managed to fix i18n just now.
Should i make a pull request to the same branch?

I added a simple select box on the default layout so you can test.

Here's the PR for it.
roiLeo#27 😄

@stephenjason89
Copy link
Contributor

Hello @roiLeo, here's the new PR for i18n. Everything seems to be working. Also migrated some webpack functions to vite.
#7045

@roiLeo
Copy link
Contributor

roiLeo commented Sep 13, 2023

Update

What's missing:

@yangwao
Copy link
Member Author

yangwao commented Sep 25, 2023

@roiLeo why is nuxt-speedkit striked?

@roiLeo
Copy link
Contributor

roiLeo commented Sep 25, 2023

@roiLeo why is nuxt-speedkit striked?

It doesn't block nuxt3 migration we can do it in a latter stage

@yangwao
Copy link
Member Author

yangwao commented Sep 26, 2023

We are thinking w/ @vikiival that we would close all outstanding PRs towards main branch and start looking forward to others pushing fixes towards nuxt branch, so we can flawlessly switch and enjoy sweet ride of Nuxt3

lmk what do you think @roiLeo as I see, there is consola and test tx issues outstanding.

I think @prury can check those TXs for now and once consola will be there, we can proceed.

@roiLeo
Copy link
Contributor

roiLeo commented Sep 27, 2023

We are thinking w/ @vikiival that we would close all outstanding PRs towards main branch and start looking forward to others pushing fixes towards nuxt branch, so we can flawlessly switch and enjoy sweet ride of Nuxt3

As you guys want, we haven't spotted all the new bugs yet. Imo we're at around 85-90% completed on Nuxt3.
I would first merge hotfix, do a release, and have more people to test/fix new app.

lmk what do you think @roiLeo as I see, there is consola and test tx issues outstanding.

consola, tests, tx, preview, download, sitemap, $seoMeta, useVModel, infinite console error in MediaItem and maybe more?

@vikiival
Copy link
Member

As you guys want, we haven't spotted all the new bugs yet. Imo we're at around 85-90% completed on Nuxt3.

Just around the time to dedicate force there ☺️

I would first merge hotfix, do a release, and have more people to test/fix new app.

So starting tomorrow unless it is not a hotfix/bugfix/p1 it will be closed

@yangwao
Copy link
Member Author

yangwao commented Sep 27, 2023

we're at around 85-90% completed on Nuxt3.

good around sweet spot to transition

consola, tests, tx, preview, download, sitemap, $seoMeta, useVModel, infinite console error in MediaItem

it's about how we will split the load with the rest of the team. I'm pretty optimistic under 2 weeks since working on those things we can run recent plugins and packages

@roiLeo
Copy link
Contributor

roiLeo commented Sep 28, 2023

Please take a break of adding new (large) features as I keep having loads of new conflicts/work

@vikiival
Copy link
Member

As agreed yesterday unless it is not a hot fix you can close it

@yangwao
Copy link
Member Author

yangwao commented Sep 28, 2023

Please take a break of adding new (large) features as I keep having loads of new conflicts/work

I was thinking today, I would

  • switch base branch to nuxt -> new PRs going there
  • merge current PRs and push new release
  • focus for the upcoming 2 weeks on making nuxt production ready
  • in case of a bigger bug, pushing from the main out
  • when - Nuxt3 - Encountered problems list #7375 will be doable short, get green light and not really bad UX, I would merge nuxt -> main
  • push it out
  • get back in flywheel as usual

@vikiival
Copy link
Member

merge current PRs and push new release

Merge fixes imo 🥹. Others can be meeged to nuxt3 later

@preschian
Copy link
Member

preschian commented Sep 29, 2023

We are thinking w/ @vikiival that we would close all outstanding PRs towards main branch and start looking forward to others pushing fixes towards nuxt branch, so we can flawlessly switch and enjoy sweet ride of Nuxt3

Please take a break of adding new (large) features as I keep having loads of new conflicts/work

So starting tomorrow unless it is not a hotfix/bugfix/p1 it will be closed

ooohhh 😮, sorry I missed this announcement. just saw it today

so, we freeze features targeting to the main branch? if said so, I can close this PR and PR

because I still see new PR targeting the main branch

@yangwao
Copy link
Member Author

yangwao commented Sep 29, 2023

Merge fixes imo 🥹. Others can be meeged to nuxt3 later

Which one?

Merged those which did not have conflicts; we can still merge once a few are resolved and do release

Meanwhile

Default branch changed to nuxt
image

so, we freeze features targeting to the main branch? if said so, I can close this #7395 and #7394

@preschian if it's not critical or p1,p2 bug, I would push it to nuxt

@yangwao yangwao changed the title Nuxt3 merging strategy Nuxt3 - merging strategy Oct 3, 2023
@yangwao
Copy link
Member Author

yangwao commented Oct 3, 2023

I would like to see a focus on

We plan to set it to production next week to have this out, as we did not have significant release updates for a while for main and I believe at the current stage it's already a major UX upgrade for the end user

@roiLeo roiLeo mentioned this issue Oct 10, 2023
@yangwao
Copy link
Member Author

yangwao commented Oct 10, 2023

It seems everything is back to main !

Nuxt3 available at https://nuxt.kodadot.xyz

Thank you to all contributors, @kodadot/internal-dev and individuals.

Let's do final checks to see if anything is missing before the release to production

telegram-cloud-photo-size-4-5848143404742066073-m

telegram-cloud-photo-size-4-5848143404742066075-x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
$$ ~51-150usd A-nuxt A-vue3 chief hacktoberfest p1 preventing everyone from using app
Projects
None yet
Development

No branches or pull requests

5 participants