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

Chelsea | MVC Tweedr #118

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Chelsea | MVC Tweedr #118

wants to merge 14 commits into from

Conversation

chelsejw
Copy link

@chelsejw chelsejw commented Apr 23, 2020

Part 1

Deliverable Submission

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5): 3.5

Completeness Level (1-5): 3

How many hours did you spend on this assignment? 8hrs

What part did you spend the most time on? Planning SQL tables, creating css

Does the assignment run without errors? If the assignment has errors, what are they?

Describe one part of the code or choose one line you would improve upon. What would you improve?

  • Show different home screen when user is not logged in
  • All users > follow button should be unfollow if user is already following them.
  • Input validation for certain things…username/display name max characters, tweets < 140 characters (wanted to use DOM manipulation to do the counter but I couldn't get the script to work)
  • Register & login should only be available if user is not logged in, logout should only be available if user is logged in

What did you think of this deliverable? Any other comments?
Enjoyed it because it got me thinking about how huge websites like Twitter actually work. I still find that I am kinda blindly doing MVC though.

@chelsejw
Copy link
Author

Part 3: Creating Hashtags

Please describe your comfort and completeness levels before submitting. Comfort Level (1-5): 2.5

Completeness Level (1-5): 2

How many hours did you spend on this assignment? 3 hrs

What part did you spend the most time on? Trying to figure out how to get a table with all the tweet info, user info, and hashtag info.

Does the assignment run without errors? If the assignment has errors, what are they? Yes (user can add hashtag when creating tweet)

Describe one part of the code or choose one line you would improve upon. What would you improve?
For one I didn't manage to display all tweets under a certain hashtag.

What did you think of this deliverable? Any other comments?
It was difficult for me to figure out what/how many queries I need to make to display all tweets of a certain hashtag

@chelsejw
Copy link
Author

chelsejw commented Apr 27, 2020

Part 4: AJAX Requests

Describe your comfort and completeness levels before submitting. Comfort Level (1-5): 2.5

Completeness Level (1-5): 1

How many hours did you spend on this assignment? 1.5hrs

What part did you spend the most time on? Writing the script.js / re-reading floobits to see what goes where. Then trying to get a hold of the correct results object. Confused as to why this.responseText (in script.js) works in the floobits demo but not in my code (I had to use request.responseText instead, where request is my XMLHttpRequest object)

Does the assignment run without errors? If the assignment has errors, what are they? Yes

Describe one part of the code or choose one line you would improve upon. What would you improve?
Would have continued onto the furthers, stopped at MVP.

What did you think of this deliverable? Any other comments?
The MVP was great for practicing AJAX requests.

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.

1 participant