Skip to content

Migrate from Next.js 14 / React 18 to Next.js 15 / React 19#2323

Merged
MikeEdgar merged 11 commits intostreamshub:mainfrom
MikeEdgar:nextjs-15
Apr 9, 2026
Merged

Migrate from Next.js 14 / React 18 to Next.js 15 / React 19#2323
MikeEdgar merged 11 commits intostreamshub:mainfrom
MikeEdgar:nextjs-15

Conversation

@MikeEdgar
Copy link
Copy Markdown
Member

@MikeEdgar MikeEdgar commented Feb 12, 2026

  • convert parameters to be promises, making server components async
  • refactoring of ResponsiveTable that removes unnecessary ResponsiveTh and ResponsiveTd components. The application does not currently utilize auto-hiding columns. This can be revisited in the future if needed.
  • remove use-resize-observer and replace with native implementation
  • Fixes to theme icon selection
  • Additional async function promise conversion
  • Update timestamp when using in-app refresh button
  • Fix cluster connection detail drawer
  • Add missing project column in cluster table when OIDC is enabled
  • Remove broken auto-refresh on group members tab, topic groups listing, and topic partitions listing

@MikeEdgar MikeEdgar added this to the 0.11.1 milestone Feb 12, 2026
@MikeEdgar MikeEdgar marked this pull request as ready for review February 12, 2026 12:04
@MikeEdgar MikeEdgar marked this pull request as draft February 12, 2026 16:39
@MikeEdgar MikeEdgar marked this pull request as ready for review February 13, 2026 15:48
@MikeEdgar MikeEdgar modified the milestones: 0.11.1, 0.12.0 Mar 4, 2026
@MikeEdgar MikeEdgar force-pushed the nextjs-15 branch 2 times, most recently from 5b02b2d to 29f9036 Compare March 5, 2026 13:29
@MikeEdgar MikeEdgar force-pushed the nextjs-15 branch 2 times, most recently from 261783e to aa5078a Compare March 6, 2026 18:09
@MikeEdgar MikeEdgar force-pushed the nextjs-15 branch 3 times, most recently from 86a040c to ee6edfa Compare March 10, 2026 19:35
@MikeEdgar MikeEdgar force-pushed the nextjs-15 branch 2 times, most recently from 5ca3390 to b13e8b5 Compare March 31, 2026 17:26
@MikeEdgar MikeEdgar marked this pull request as ready for review March 31, 2026 19:52
@MikeEdgar MikeEdgar force-pushed the nextjs-15 branch 3 times, most recently from 2ccf4a5 to 1373448 Compare April 4, 2026 10:49
@MikeEdgar MikeEdgar added this to the 0.12.1 milestone Apr 4, 2026
@MikeEdgar MikeEdgar requested a review from a team April 6, 2026 15:07
MikeEdgar and others added 10 commits April 7, 2026 07:16
Signed-off-by: Michael Edgar <medgar@redhat.com>
Co-authored-by: hemahg <hhg@redhat.com>
Co-Authored-By: dependabot[bot] <support@github.com>
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Assisted-by: Claude Code (Sonnet 4.5) <noreply@anthropic.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
- Fixes to theme icon selection
- Additional async function promise conversion
- Update timestamp when using in-app refresh button
- Fix cluster connection detail drawer
- Add missing project column in cluster table when OIDC is enabled
- Remove broken auto-refresh on group members tab, topic groups listing,
and topic partitions listing

Signed-off-by: Michael Edgar <medgar@redhat.com>
Assisted-by: Claude Code (Sonnet 4.5) <noreply@anthropic.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
Signed-off-by: Michael Edgar <medgar@redhat.com>
@MikeEdgar
Copy link
Copy Markdown
Member Author

/systemtests run

Copy link
Copy Markdown
Contributor

@jankalinic jankalinic left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

I will run those tests locally as well before giving an approve.

If the playwright assertions work reliably for every / most of the cases, we could refactor the STs to save exec time. Also thanks for the related async update for kubetest4j!

oldOlmChannel: 0.12.x
newOlmChannel: alpha
oldOlmChannel: 0.11.x
newOlmChannel: 0.12.x
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't we always go for the latest => alpha? Or has that channel versioning/naming changed?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The problem here is that the current build is 0.12.1-snapshot, which is in 0.12.x already. So the initial test for 0.12.0 fails because it automatically updated to 0.12.1-snapshot. I think we need to keep use the version-specific channels here to prevent the current snapshot from getting installed right away.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

So would you recommend using 0.11 -> latest? I'm thinking about how OLM is skipping/upgrading to those versions. I'm fine with version channels, just thinking if there are any cases in which this might cause future issues.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

You mean 0.11.x to alpha instead of 0.11.x to 0.12.x? At the moment that seems like it would work, but there's only one way to find out 😆

Signed-off-by: jkalinic <jkalinic@redhat.com>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

@jankalinic
Copy link
Copy Markdown
Contributor

/systemtests run --testcase=KafkaST

@streamshub-bot
Copy link
Copy Markdown

✅ Systemtests run finished - success ✅

Test Summary:

  • TOTAL: 3
  • PASS: 3
  • FAIL: 0
  • SKIP: 0

Used parameters:

  • TEST_CASE: KafkaST
  • PROFILE:
  • INSTALL_TYPE: olm
  • RETRY_COUNT: 0
  • ENVS:

Copy link
Copy Markdown
Contributor

@jankalinic jankalinic left a comment

Choose a reason for hiding this comment

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

Thank you so much for this PR!

@MikeEdgar
Copy link
Copy Markdown
Member Author

@alexcreasy this needs to be merged and 0.12.1 released, but please still give your feedback on the changes in case you find anything we can fix in the next patch 🙏

@MikeEdgar MikeEdgar merged commit ed0a83e into streamshub:main Apr 9, 2026
8 checks passed
@MikeEdgar MikeEdgar deleted the nextjs-15 branch April 9, 2026 19:43
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.

4 participants