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

Our Umbraco: High five feed #277

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

Our Umbraco: High five feed #277

wants to merge 80 commits into from

Conversation

crgrieve
Copy link
Contributor

@crgrieve crgrieve commented Jul 8, 2018

High Five feed worked on during CG18 retreat (with Niels, Mike and Kyle).

Adding a feature providing a way to high five someone for giving something to Umbraco community: article, help,video, PR etc. This form and feed of recent high fives is on home page when you are logged in. Only logged in users can high five.

High five submit form has a user search: type in >3 letters of the name and it will give you a drop down of matching users. Then you can select an activity type (blog post for example) then provide a link. The page will dynamically build up a preview of the high five including fetching the page title of the link.
Here is a video of the form: https://drive.google.com/file/d/1ws0SHJd5XSvLXgopApUIIHX39GSj1Uk1/view

We've also added a feed of users high fives to their profile.

Ideas for future:

  • Should we add caching (clearing it when a new high five submitted)?
  • notification when someone high fives you.
  • full page of high five feed (rather than just most recent on home page?).
  • embed link for blog posts so we can have people high five to our from the original post.

hartvig and others added 30 commits May 20, 2018 15:36
Needs work for responsive styling
…laying most recent high fives, with a mock API response.
…ionality as those libraries exist already on Our.Umbraco.
cssquirrel and others added 29 commits May 21, 2018 17:05
hopefully more efficient :)
… required, which is great for accessibility.
…for url input, add more compliments from retreat.
refactoring & comments.
resetting preview so it doesn't remember the last one when you try to submit another.
@nul800sebastiaan nul800sebastiaan changed the base branch from master to main July 15, 2020 13:25
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.

4 participants