Skip to content
This repository has been archived by the owner on Sep 5, 2023. It is now read-only.

Proud Puffins - submitting final PR that fell through the cracks. #138

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

babarrett
Copy link
Contributor

All these changes were done before the deadline.
Some improve the Docs / how to install.
Some are code fixes that are needed to make the project work.
All files are time-stamped before the deadline.

chill and others added 15 commits August 9, 2020 07:22
- Remove '\\n' s and lowercase entire bio
- Remove swear words, other web links for privacy concerns
- Remove user_gen.py, raw.json from tools/
- Remove name mentions
- likedyou and youliked were returning UserVote objects which made Null Qset when intersected
- Made them return values_list of users and intersected them to actually give matches
- Passed them into a matches list which gets users based on pk's of the intersected list
Add requirements
- Add alternative way to install the application
- Add instructions to use the app.
Add a note about lint error
resize headings of Learning materials and normalize.css to match rest of them
@ghost ghost added the needs 2 approvals label Sep 12, 2020
@babarrett babarrett changed the title Prwoud Puffins - submitting final PR that fell through the cracks. Proud Puffins - submitting final PR that fell through the cracks. Sep 12, 2020
Base automatically changed from master to main March 13, 2021 19:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants