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

Cancel button + noscore not in edit #292

Merged
merged 2 commits into from
May 23, 2024
Merged

Conversation

Aqua-sc
Copy link
Contributor

@Aqua-sc Aqua-sc commented May 23, 2024

  • Ge kunt nu uw feedback edit cancellen en dan neemt het gwn terug de oude waarde
  • Als ge op edit score drukt zo met da potloodje en er is geen score ist edit veld nu gewoon leeg

@Aqua-sc Aqua-sc requested a review from badduck32 May 23, 2024 11:17
@Aqua-sc Aqua-sc marked this pull request as ready for review May 23, 2024 11:25
scriptPlaceholder = "bash /shared/input/helloworld.sh > \"/shared/output/helloWorldTest\n"+
"bash /shared/input/helloug.sh > \"/shared/output/helloUGent\n"
scriptPlaceholder = "bash /shared/input/helloworld.sh > \"/shared/output/helloWorldTest\"\n"+
"bash /shared/input/helloug.sh > \"/shared/output/helloUGent\"\n"
Copy link
Contributor

Choose a reason for hiding this comment

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

dit lijkt mij per ongeluk aangepast? Want dit heeft niet echt iets te maken met de cancel knop en de "No score" in de submission feedback

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah da was gwn nog een foutje in de placeholder vd template. Kdacht da rechtstreeks op frontend gepushed had ma blijkbaar zat ik nog op deze branch

Copy link
Contributor

Choose a reason for hiding this comment

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

ok dan is alles goed, ik zal mergen

@badduck32 badduck32 merged commit 81ebeea into frontend May 23, 2024
1 check passed
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.

2 participants