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

Dashboard Theme #205

Open
MatthewVita opened this issue Sep 13, 2015 · 42 comments
Open

Dashboard Theme #205

MatthewVita opened this issue Sep 13, 2015 · 42 comments
Assignees

Comments

@MatthewVita
Copy link
Contributor

make sure the RoR dashboard theme is extremely similar to that of our regular webapp.

@MatthewVita MatthewVita self-assigned this Sep 13, 2015
@MatthewVita MatthewVita added this to the Admin Dashboard Epic milestone Sep 13, 2015
@MatthewVita
Copy link
Contributor Author

first pass :) needs a lot of work but at least rails dashboard side is wired up a bit better now!

@timothyfcook look how beautiful:

1
2
3
4
5

@timothyfcook
Copy link
Contributor

oooo pretty! 100x improvement. Time for @cameronscott137 and @chelseaerdner and @dmtroyer to do some magic and bring it home.

Remember, we want to have a place where you can build maps as well. I'll make a new ticket for thinking about what this looks like...

@MatthewVita
Copy link
Contributor Author

yep. still a lot of work to be done!

@chelseaerdner
Copy link

Hello everyone,

So I took a pass at styling these pages and creating a sitemap to confirm that my understanding of this flow is correct. Steve and Matthew gave me access to see what was going on and I want to confirm some things.

Here is the flow I mapped out. From the homepage, a user can explore opportunities, join now, or login. The user only have to create an account once therefore, only identifying if they are a learner or organization one time. After that, they simply login with the credentials they created.

If a user that does not have an account is exploring opportunities and wants to participate in one, they can click register but it will just redirect them to the signup page. From there they will choose Organizer or Learner and create and account.

screen shot 2015-10-28 at 12 35 19 pm

Moving on from that...I styled some of the screen Matthew posted above.

Here is the first screen a user sees when creating and account:

screen shot 2015-10-28 at 12 51 55 pm

If they chose organizer, this is what they would see:

screen shot 2015-10-28 at 12 53 03 pm

If they chose learner, this is what they would see:

screen shot 2015-10-28 at 12 54 27 pm

If a user already has an account and choses to login this is what they would see:

screen shot 2015-10-28 at 12 55 20 pm

I am going to working on the individual dashboard screens now. Although, I am lack context into some of it. I will reach out if I run into any issues.

As for the screens above, let me know thoughts and opinions.

Thanks!
Chelsea

@dmtroyer
Copy link
Contributor

Wow, I think the styling looks great!

I think there are aspects of the admin dashboard that probably need a bit more thought than styling what is currently there. For example:

  • How do we make opportunity and opportunity instance creation more intuitive? Possibly obfuscating the two from the user to make it less confusing.

@timothyfcook were you hoping to discuss some of this on Friday?

@chelseaerdner
Copy link

Thanks, @dmtroyer!

Makes sense though, I will hold off until our conversation Friday to move forward.

@MatthewVita MatthewVita modified the milestones: November 3rd Demo Epic, Admin Dashboard Epic Oct 28, 2015
@MatthewVita
Copy link
Contributor Author

Github doesn't allow for 2 people to be assigned to an issue, but consider Chelsea and David the assignees for this. I put this in the "Nov. 3rd Demo Epic" milestone, but that doesn't mean it must be completed by then. Just try to get as much done as you can.

@timothyfcook, do you agree?

@timothyfcook
Copy link
Contributor

Definitely. We'll finalize the design on Friday and then lean on @cameronscott137 to implement. We very well might have it done before the Nov. 3rd deadline.

@dmtroyer
Copy link
Contributor

Some brainstorming thoughts from @timothyfcook and I:

  • Opportunities & Opportunity Instances
    • Opportunity page lists the associated Opportunity Instances
    • Break out to another page to add a new or edit an Opportunity Instance.
      • Matching fields are prepopulated from the parent Opportunity
      • Link to create a new venue
      • Drop down for choosing Topic allows for creating a new one.
  • Venues
  • User Settings
    • Manage associated Organizers
      • Edit existing Organizers the user is associated with
      • Create new Organizer
    • Request to join an already existing Organizer (Drop-down or search to choose)
  • Style
    • Move to a vertical menu with icons
      img_20151030_101220

@timothyfcook
Copy link
Contributor

Great chat IRL with @chelseaerdner today. Next steps are for her to:

  • mock-up revisions for onboarding flow
  • input flow for the opportunity builder
  • design layout for all admin dashboard pages

Note: we talked about using a variable for tenant_name in every place where "Learn.Pgh" would show up so that future tenants can easily change them via a config file or Super Admin dashboard, depending on what we decide for #285

@dmtroyer
Copy link
Contributor

I made an issue to start enumerating tenant-specific config on #316

@chelseaerdner
Copy link

Happy Friday everyone,

I wanted to share my progress with the team in regards to the revisions to onboarding and the admin dashboard pages this week.

Onboarding Flow:
Last week we determined that an organizer needs to identify themselves as an individual organizer or one that is associated with an organization. Individual organizers provide a little information about their work. Organization organizers can either join an existing org or create a new one. The first user to create an organization is automatically the admin of that organization and all other users associated with it are "organizers."

caac_admin_11062015-04

caac_admin_11062015-07

caac_admin_11062015-08

Admin Dashboard:
There are a lot of things to think about with the admin dashboard screens but I started to flush out the different aspects a user sees when first logging in and landing on their dashboard. I think the 5 buckets are: Overview, Opportunities, Locations (or Venues), Organizers, and Settings. The screens below are just the high level views of: Opportunities, Locations, and Organizers. I am still working through the rest early next week. I just saw the issue about replacing Locations with Venues and I will incorporate that next week also.

caac_admin_11062015-10

caac_admin_11062015-09

caac_admin_11062015-12

There is a lot to look at here so please let me know if I can clarify any questions or issues that come up!

Thanks!

@chelseaerdner
Copy link

Also, I briefly spoke with @sdanko11 about using another word for instances within an opportunity. I think that the word instances is a little too technical for the users within orgs. Does anyone have a user friendly suggestion to replace it on the front-end? Steve and I were thinking something like...occasions or occurrences? Basically we need a word that displays there are multiple events within an opportunity.

@dmtroyer
Copy link
Contributor

dmtroyer commented Nov 6, 2015

+1 on thinking of another word for opportunity instances.

This seems like a good time to consider moving the admin dashboard from the rails app to the angular app. There are certainly costs associated with that move which need to be weighed against the cost of maintaining these separately when they really seem to be converging. cc: @MatthewVita @sdanko11 @cameronscott137 @timothyfcook @whit537

@MatthewVita
Copy link
Contributor Author

@chelseaerdner the mockups look wonderful.

@dmtroyer #342

@MatthewVita MatthewVita removed this from the November 3rd Demo Epic milestone Nov 7, 2015
@chelseaerdner
Copy link

@MatthewVita Thank you! I will post more progress as I complete it this week.

@dmtroyer
Copy link
Contributor

So what's the strategy for moving forward with this in light of decisions and findings on #342 and #344? @timothyfcook suggested @cameronscott137 and myself pairing, which him doing the frontend (angular) and me doing the backend (rails).

If we break it out into user stories/issues/prs by feature, I think that could be a good way to start? I think a first candidate could be user registration, and we have mockups for that!

@MatthewVita
Copy link
Contributor Author

@dmtroyer I've created #347 for this since both ends have to be developed in tandem. I can join in on this as well!

@timothyfcook: note that Angularizing the dashboard is really a technical debt item that is non-trivial. Also note if we jump on it now, we're not following the currently roadmap properly:

capture

We are currently around the red arrow. The blue arrow represents where we handle the bulk of our technical debt (such as this).

tl;dr: I love improving our stack as much as everyone else does but we are all responsible for following our roadmap appropriately. You'll need to decide if we should stick to the current roadmap or adjust it.

@dmtroyer
Copy link
Contributor

@MatthewVita thank you for pointing this out re: the road map. I think this is really critical for a project such as this where individuals are distributed but trying to work towards the same goal.

I also think it is necessary to weigh that by implementing the proposed improvements to the admin ui in rails, we will incur even further technical debt and it would make angularizing the admin side of things even more costly and less worthwhile down the line. There's a whole lot of work to be done for the admin ui either way, but I agree this should be considered out of step with the roadmap in some sense.

@MatthewVita
Copy link
Contributor Author

@dmtroyer I totally agree! I'm just being a big stickler about the roadmap and it's ultimately up to Tim to decide priorities with it.

📈📊📈📊📈📊

@timothyfcook
Copy link
Contributor

Great conversation, thanks for raising this.

I agree that we shouldn't pursue tech debt issues unless really necessary for the MVP.

The main question to help decide this is:

  • Can we adequately create an Admin UI that we feel good about for the MVP launch without moving the Admin side over to Angular? I'm fine with leaving it, but also don't want to waste resources on a Rails Admin UI if we are just going to move it later.

Thoughts?

@dmtroyer
Copy link
Contributor

Can we adequately create an Admin UI that we feel good about for the MVP launch without moving the Admin side over to Angular? I'm fine with leaving it, but also don't want to waste resources on a Rails Admin UI if we are just going to move it later.

@MatthewVita and @cameronscott137 might have a better idea of this, but my gut feeling is that it would be doable but ugly (keeping it in rails). My main concern is how we would go about maintaining the unified user experience, like the headers and navigation, in what @chelseaerdner is cooking up. Maybe we could add an iframe to the rails views inside the angular views or just duplicate it and maintain them separately.

In all, I feel that It would save us some time now in that we wouldn't have to implement token auth and the necessary rails side web services, but we're creating an order of magnitude more tech debt by doing it in rails now if we ever want to move it to angular in the future.

@MatthewVita
Copy link
Contributor Author

From a technical perspective: what @dmtroyer said.

From a planning perspective: we've already scoped out tech debt on our roadmap for post-MVP mainly in terms of adding unit tests and creating great documentation (filter against "tech debt" label in our issues). Do we add Angular dashboard tech debt to that list or tackle it now? It will delay the MVP a bit (not for a bad reason, of course).

From a project perspective, the sole purpose of the MVP is to quickly*** get it in front of educators/learners/cities/non-profits to see if our platform provides value and if we need to pivot based off of that feedback. I'm sure we didn't get everything right with this first iteration and that's okay. Worst case scenario is that we do pivot and the technical investment in the MVP dashboard was all for nothing.

*** I hate this word, but you know what I mean. I'm touching on the "fail fast" model.

@dmtroyer
Copy link
Contributor

::nudge::

@MatthewVita
Copy link
Contributor Author

@dmtroyer
Copy link
Contributor

I believe @timothyfcook is somewhere in the throws of having a baby! 🚼

@MatthewVita
Copy link
Contributor Author

Okay, @whit537, @dmtroyer, @sdanko11, and @cameronscott137. Since @timothyfcook is out (#353 (comment)), let's make an executive decision (via a vote) so we can move things forward. Just edit the following comment with your vote:

@MatthewVita
Copy link
Contributor Author

Should we move forward with the Angular dashboard to have a more robust solution or stick with Rails serving up the client to get the MVP done quicker?

NAME VOTE (angular/rails)
Matthew angular
David angular
Cameron null
Steve null
Chad angular

EDIT: oops, how could I forget to add Chad!?

@dmtroyer
Copy link
Contributor

@chelseaerdner could we get some assets of the icons (svgs?) from your work in #205 (comment)? Thanks.

@chadwhitacre
Copy link
Contributor

EDIT: oops, how could I forget to add Chad!?

@MatthewVita :-)

Happy to follow along with Angular if you and @dmtroyer wanna lead the charge. New territory for me. :-)

@MatthewVita
Copy link
Contributor Author

Great. @dmtroyer, @whit537: Let's be very disciplined about making small user/technical stories in https://github.com/saxifrage/cityasacampus/milestones/Angularize%20Admin%20Dashboard%20Epic so we don't step on each other's feet.

@MatthewVita
Copy link
Contributor Author

@dmtroyer
Copy link
Contributor

Let's be very disciplined about making small user/technical stories in https://github.com/saxifrage/cityasacampus/milestones/Angularize%20Admin%20Dashboard%20Epic so we don't step on each other's feet.

Ok cool. I think what I will do is pare back the initial PR to login and stripping out the old dashboard. I think that will give us the best launching pad for parallel work.

also, do you guys mind if I just merge everything in

Go for it!

@chelseaerdner
Copy link

@dmtroyer working on saving those icons out now! Should have them over shortly.

@dmtroyer
Copy link
Contributor

@chelseaerdner sent me the assets! Here's a link since github doesn't allow .svg or .zip:

https://drive.google.com/file/d/0Bwt4WeLO9DW9QmE5Q2VtWk5TU1U/view?usp=sharing

@chadwhitacre
Copy link
Contributor

Actually, I just saw an SVG over at https://github.com/oh-my-fish/oh-my-fish/. O.O

Looks like a recent (re)addition on GitHub's part? oh-my-fish/oh-my-fish#139 just happened yesterday.

@chadwhitacre
Copy link
Contributor

Boo! :-(

screen shot 2015-11-30 at 7 36 45 pm

@chadwhitacre
Copy link
Contributor

Here's my first test with http://recordit.co/ ...

ja7agffipa

@chadwhitacre
Copy link
Contributor

Smaller ...

errzfmwkn7

@dmtroyer
Copy link
Contributor

dmtroyer commented Dec 1, 2015

image

@chadwhitacre
Copy link
Contributor

Woo-hoo!

!m @dmtroyer @chelseaerdner

@chelseaerdner
Copy link

Looking good! @dmtroyer (and everyone)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants