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

[1.20.1 Forge] restarts up to 3 times, popup notification for user to restart client upon an launch sequence update case not showing up #144

Closed
MrKashew opened this issue Jul 25, 2023 · 11 comments
Labels
bug Something isn't working

Comments

@MrKashew
Copy link

MrKashew commented Jul 25, 2023

Motivation

For users not viewing Console such as
[20:27:33] [modloading-worker-0/INFO] [AutoModpack/]: Restart your client!

Description

For this case some sort of UX notification
[24/07/2023 22:39:23 PM] [modloading-worker-0/INFO] Restart your client!

Other information

No response

@Skidamek
Copy link
Owner

There is actually maybe it doesn't shows on forge somehow

@MrKashew
Copy link
Author

MrKashew commented Jul 25, 2023

yep this is on Forge 1.20.1, referring to the launch "restart client" case not on the join server "restart client" case

@MrKashew MrKashew changed the title Feature Request: popup notification for user to restart client upon an launch-autoupdate [ 1.20.1 Forge - Feature Request] popup notification for user to restart client upon an launch sequence update case Jul 26, 2023
@Skidamek Skidamek changed the title [ 1.20.1 Forge - Feature Request] popup notification for user to restart client upon an launch sequence update case [ 1.20.1 Forge] popup notification for user to restart client upon an launch sequence update case not showing up Jul 26, 2023
@Skidamek Skidamek added the bug Something isn't working label Jul 26, 2023
@Nosonii
Copy link

Nosonii commented Jul 27, 2023

[Minecraft 1.20.1 AutoModpack 3.4.3 Forge]

I am having this problem too, after I synced my client to my server and restarted, every time I launch the game AutoModpack will force close the game after all the mods are loaded with exit code 0, and when disabling AutoModpack the game will be able to launch.

I suspected this may have been due to a mismatch of having client-side mods that are not present in the server's mod directory, so I made the directory /automodpack/host-modpack/mods/ and put ALL the mods in there but the force closing issue is still there after I synced again.

@Skidamek
Copy link
Owner

@Nosonii oh what, so that's a different not releated issue, please open new issue

@MrKashew
Copy link
Author

MrKashew commented Jul 27, 2023

is this flow expected or is this part of bug
start client -> client updates, force close (no popup) -> restart client again -> client logs same updates, force close (no popup) -> restart client (success)

@Skidamek
Copy link
Owner

Skidamek commented Jul 27, 2023

is this flow expected or is this part of bug
start client -> client updates, force close (no popup) -> restart client again -> client logs same updates, force close (no popup) -> restart client (success)

Definitely no, it should be like that:

start client -> client updates -> show popup -> click button on popup -> force close game -> manually start game agian successfully

@Skidamek
Copy link
Owner

Skidamek commented Jul 27, 2023

I guess it's an issue because forge loads mods in parallel and somehow it doesn't work there

@MrKashew
Copy link
Author

gotcha sum of bugs here for future reference:

  • forge has weird behavior where if update happens in launch:
    • start client -> client updates, force close (no popup) -> restart client again -> client logs same updates, force close (no popup) -> restart client (success)
    • should be start client -> client updates -> show popup -> click button on popup -> force close game -> manually start game agian successfully
  • pop up does not show on force close
  • forge does not have the toast messages as seen in video

@Skidamek
Copy link
Owner

Skidamek commented Jul 27, 2023

Well the video is quite a lot outdated so the toasts aren't a thing anymore because they were quite buggy. Now are just much more responsive screens instead, which I think are just better. I have plans to re-record that agian somewhere in the near future.

@MrKashew MrKashew changed the title [ 1.20.1 Forge] popup notification for user to restart client upon an launch sequence update case not showing up [ 1.20.1 Forge] popup notification for user to restart client upon an launch sequence update case not showing up, restarts up to 3 times Jul 28, 2023
@MrKashew MrKashew changed the title [ 1.20.1 Forge] popup notification for user to restart client upon an launch sequence update case not showing up, restarts up to 3 times [ 1.20.1 Forge] restarts up to 3 times, popup notification for user to restart client upon an launch sequence update case not showing up Jul 28, 2023
@Skidamek Skidamek changed the title [ 1.20.1 Forge] restarts up to 3 times, popup notification for user to restart client upon an launch sequence update case not showing up [1.20.1 Forge] restarts up to 3 times, popup notification for user to restart client upon an launch sequence update case not showing up Jul 28, 2023
@Skidamek
Copy link
Owner

Skidamek commented Jul 28, 2023

@Skidamek
Copy link
Owner

Skidamek commented Aug 2, 2023

Fixed in 3.4.4

@Skidamek Skidamek closed this as completed Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants