search

JennieHo-HPE / Sapporo

Web App for hosting coding competition
0 stars 0 forks source link

[bug] Update/Remove the versions in language selection #6

Closed JennieHo-HPE closed 2 years ago

JennieHo-HPE commented 2 years ago

image

Rust 1.56 Golang 1.16

JennieHo-HPE commented 2 years ago

https://www.hpcodewars.com.tw/rules Java 8 (OpenJDK 8u312) Python 2.7 Python 3.10 C (gcc11.2) C++ (gcc11.2) Rust 1.56 Golang 1.16

Swift 5.5

11/17

Java 8 Python 2.7 Python 3.10 C C++ (C++14/17) Rust 1.56 Golang 1.16 Swift 5.5

JennieHo-HPE commented 2 years ago

https://github.com/JennieHo-HPE/Sapporo/blob/master/imports/ui/admin/dockerConfig.jsx#L62

yilanhung commented 2 years ago

NOT A BUG. That is the default list built into the system for convenience. You always have to manually review and "configure" the list every year no matter what the default says.

yilanhung commented 2 years ago

So if you agree too, @JennieHo-HPE, close this issue as no-fix.

# I'd kinda like this 'default list' to be populated from a config value, not hard-coded, but that'd a new feature request.

JennieHo-HPE commented 2 years ago

image

The C++ 11 is supposed to be C++ ? or we may seperate to C++14 and C++17

yilanhung commented 2 years ago

@JennieHo-HPE The admin has to update the language list. I believe what Jerry did was just populating the default list. You can do that if you want.

image

yilanhung commented 2 years ago

@JennieHo-HPE Yeah, I just updated "C++ 11" and made it "C++ 14". Also added "C++ 17". (logged in to the testing system as admin)

image

Looks like the language options on the editor screen appear in an order as they were entered by the admin. To get around that defect, we'll have to re-create the list in a sane order.

Or do you want to see it as a bug?

JennieHo-HPE commented 2 years ago

In end user's view, it's a bug in 2021. However, the Sapporo is good We may record this kind of items to 'regular task list' for every year.