Skip to content

Add bug issue form#5603

Open
MightyFilipns wants to merge 3 commits intocuberite:masterfrom
MightyFilipns:bug-issue-form
Open

Add bug issue form#5603
MightyFilipns wants to merge 3 commits intocuberite:masterfrom
MightyFilipns:bug-issue-form

Conversation

@MightyFilipns
Copy link
Member

Add the bug issue form
Identical to the existing issue template
Issue forms create an interactive web interface instead of just a big markdown input box
Fixes #5427

Copy link
Member

@12xx12 12xx12 left a comment

Choose a reason for hiding this comment

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

Great idea!

I have two notes and a question: Is there a way to preview this?

Comment on lines +69 to +72
- Windows
- Linux
- macOS
- BSD
Copy link
Member

Choose a reason for hiding this comment

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

Can we add "other" to open up a text box?

Copy link
Member Author

Choose a reason for hiding this comment

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

It looks like its not possible

Copy link
Member

Choose a reason for hiding this comment

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

hm. any ideas on how to circumvent this?

Some people try to run cuberite on weird OS. like their routers xD

Copy link
Member Author

Choose a reason for hiding this comment

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

The best we can do is to add an optional input box with a label that says to only fulfil if Other is chosen as the OS option

Comment on lines +49 to +59
- 1.12.2
- 1.12.1
- 1.12
- 1.11.1 (1.11.2)
- 1.11
- 1.10.X
- 1.9.4 (1.9.3)
- 1.9.2
- 1.9.1
- 1.9
- 1.8.X
Copy link
Member

Choose a reason for hiding this comment

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

Can we unify the version layout?

either 1.12.x or 1.12, 1.12.1, 1.12.2 and so on

Or is there a reason to do it this way?

Copy link
Member Author

Choose a reason for hiding this comment

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

The reason for 1.10.X and 1.8.X is that all minor versions share the same protocol version. Meaning that a bug that affects one should affect all of them.

Copy link
Member

Choose a reason for hiding this comment

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

I see.

maybe add (ProtocolVersion xxx) at the end so people in the future know the reason

Copy link
Member Author

Choose a reason for hiding this comment

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

I have added protocol version numbers to avoid future confusion

@MightyFilipns
Copy link
Member Author

To preview click View file on the bug_report.yml

Unify version layout
Add "Other" as an OS option
Add protocol version numbers
Add missing versions
@bearbin bearbin closed this Aug 7, 2025
@bearbin bearbin reopened this Aug 7, 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.

Update Issue Template

3 participants