Skip to content
This repository has been archived by the owner on Jul 23, 2022. It is now read-only.

Why are exercises 4 & 5 separated? Consider consolidating. Also consider just removing OWASP. #601

Open
jacobsee opened this issue Jan 20, 2021 · 2 comments · Fixed by #608
Labels
bug Something isn't working enhancement New feature or request technical

Comments

@jacobsee
Copy link
Member

Problem:
It is awkward to introduce and talk all about a tool (Ex4) that they’re not going to be able to actually use until later (Ex5). Both exercises are pretty short, especially if the other suggestion of moving Ex4 pt2 to Ex2 is taken. Additionally, I don't like OWASP here as its discussion adds little value and it doesn’t even work in the pipeline.

Proposed solution:
Crunch em together. Cut OWASP out (or fix).

@jacobsee
Copy link
Member Author

jacobsee commented Feb 1, 2021

Reopening - automatically closed by PR merge but we still need to talk about separation of 4 & 5.

@jacobsee jacobsee reopened this Feb 1, 2021
@springdo
Copy link
Contributor

springdo commented Feb 1, 2021

Hey @jacobsee - I think these two exercises were trimmed down over previous changes made to the course which is why they seem odd to separate.

Bt we've always introduced them at the same time, ie doing the bigpicture updates and such. And I guess more importantly they both have separate Star Wars Crawls (with the 4th one being my absolute favourite!) 💪

@eformat eformat added bug Something isn't working enhancement New feature or request technical labels Feb 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement New feature or request technical
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants