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

Changes to the startcodes of adventures for better slides. #4332

Closed

Conversation

MarleenGilsing
Copy link
Collaborator

@MarleenGilsing MarleenGilsing commented Jun 8, 2023

Note to self that I still have to make these changes in Dutch:

In het NL nog aanpassen:

Level 4: default voorbeeldcode is startcode
quotation marks intro voorbeeldcode = startcode en toevoeging ‘from now on’

Level 6 : maths voorbeeldcode = startcode
Is_command toevoegen met startcode die nu vbc is.
Calculator adventure voorbeeldcode is startcode

Level 9: if los avontuur maken met een if in een if.

Level 10: dishes vbc wordt startcode
Level 11 calculator vbc wordt startcode

Level 14 default:
answer = ask 'Do you want to learn more?'
if answer == 'yes'
print 'Go to the next adventure!'
if answer != 'yes'
print 'That is a shame!'

Level 16 random vbc to startcode

Note to self that I still have to make these changes in Dutch:

In het NL nog aanpassen:

Level 4: default voorbeeldcode is startcode
 quotation marks intro voorbeeldcode =  startcode en toevoeging ‘from now on’

Level 6 : maths voorbeeldcode = startcode
Is_command toevoegen met startcode die nu vbc is.
Calculator adventure voorbeeldcode is startcode

Level 9: if los avontuur maken met een if in een if.

Level 10: dishes vbc wordt startcode
Level 11 calculator vbc wordt startcode

Level 14 default:
answer = ask 'Do you want to learn more?'
if answer == 'yes'
    print 'Go to the next adventure!'
if answer != 'yes'
    print 'That is a shame!'

Level 16 random vbc to startcode
@ghost
Copy link

ghost commented Jun 8, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@Felienne Felienne marked this pull request as draft June 12, 2023 17:56
@Felienne
Copy link
Member

Ik heb m eveb op draft gezet, zodat je je note to self niet vergeet :)

@MarleenGilsing
Copy link
Collaborator Author

Ik was doorgegaan op deze Slides branch om deze aanpassinkjes nog even door te voeren, maar zie nu dat de Nederlandse avonturen daar nog allemaal weblate engels zijn. Op main staat alles wel vertaald in het Nederlands, dus ik denk dat het het slimst is om het nu opnieuw te doen met een nieuwe branch vanaf main, want anders zijn die avonturen straks niet meer vertaald.

@Felienne
Copy link
Member

Ik was doorgegaan op deze Slides branch om deze aanpassinkjes nog even door te voeren, maar zie nu dat de Nederlandse avonturen daar nog allemaal weblate engels zijn. Op main staat alles wel vertaald in het Nederlands, dus ik denk dat het het slimst is om het nu opnieuw te doen met een nieuwe branch vanaf main, want anders zijn die avonturen straks niet meer vertaald.

Ha Marleen! Dat hoeft niet perse op een nieuwe branch, ik kan dat zo wel in orde maken, ben over een half uurtje op de VU en dan kijk ik even met je mee.

@MarleenGilsing
Copy link
Collaborator Author

@Felienne Deze bevat toch nog wel wat veranderingen die nog niet op main staan, dus deze mag nog doorgevoerd.

Als dat is gedaan ga ik even checken of Nederlands en Engels nog helemaal overeenkomen met startcodes, want nu vergelijkt het een beetje lastig, dus die zet ik op mn to-do lijstje.

@MarleenGilsing
Copy link
Collaborator Author

@Felienne I don't know why it failed. Could you check it, please?

@Felienne
Copy link
Member

@Felienne I don't know why it failed. Could you check it, please?

hmmmmm what a weird issue! I will fix the conflicts and see if that resolves anything. Let's try to merge this Thursday or we will keep running into issues.

@Felienne
Copy link
Member

Ok, sorry @MarleenGilsing, I just checked but there are too many changes per file to really merge this (20+ for multiple files). So I think the best part forward is to close this and to redo the changes on a fresh PR, sorry 😭

@Felienne
Copy link
Member

Closed, moved almost all changes to #4553

@Felienne Felienne closed this Sep 26, 2023
@Felienne Felienne deleted the Changing-some-start-codes-because-of-the-slides branch November 23, 2023 20:27
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