ResetNetwork / apply-app

The app behind the ResetNetwork
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

We want a "Partners" page that looks like Luminate's #79

Closed danblah closed 3 years ago

danblah commented 4 years ago

see https://luminategroup.com/portfolio.

scott-psycle commented 4 years ago

@danblah We'll start to have a look at spec'ing the details for this new page type. A couple of initial questions though.

  1. Am I correct that you want to have a list page for ALL investments and also an individual Partner page type which gives details of a specific partner organisation, similar to this on Luminate? https://luminategroup.com/investee/africa-check

  2. On Luminate we have a couple of entities:

An Organisation can have multiple investments associated with it. Will this be the case with Reset? Or will it be 1:1?

I'll come back to you to confirm specifics about data fields and values once we've had a chance to discuss.

danblah commented 4 years ago
  1. Yes.
  2. Yes.

Exactly.

danblah commented 4 years ago

Also not for MVP but so we don't make it in a way that makes it not possible... Along with the ability to manually create organizations and add projects/investments to them, we're going to want the ability to create a new project/investment from approved applications (or associate a new application to an existing organization as a new investment) in an automated way.

danblah commented 4 years ago

Bump bump on this. Can we get an update and some progress here?

Parbhat commented 4 years ago

@danblah Is this issue ready to work on? If yes and no one is working on it, @ankitjavalkar can start work on it.

frjo commented 4 years ago

@Parbhat Yes, this is ready to work on.

Parbhat commented 4 years ago

Thanks @frjo. @ankitjavalkar You can go through the issue and ask questions if need more details in requirements.

frjo commented 4 years ago

@Parbhat @sks444 Some parts of this might be time-consuming. Reset would not mind a simpler version with the basic functionality as a start. Extra features can then be added in future work.

The two content types with a minimum set of fields and a table view is a good start. Maybe possible to reuse part from "/apply/submissions/all/" able view?

sks444 commented 4 years ago

Thanks for the updates @frjo. I will look into the table implementations done for /apply/submissions/all/ and try to use the similar for this feature.

sks444 commented 4 years ago

@frjo, wanted to confirm a few things before proceeding with the implementations:

Organization:
    - name 
    - status: current partner, alumni partner
    - type: profit, non-profit
    - description: a small description about the partner
    - logo-url: partner logo
    - web-url: website url of the partner

Investment:
    - name
    - primary impact area: a choice field with choices(Civic Empowerment, Data & Digital Rights, Financial Transparency, Independent Media, Reset, Other)
    - geographic focus: a choice field with choices(Africa, Central & Eastern Europe, Global, Latin America, Multi-Country, Southeast Asia, United States, Western Europe, other)
    - year
    - amount committed(in USD)
    - type
    - purpose
    - description
frjo commented 4 years ago

@danblah Can you answer the questions above?

sks444 commented 4 years ago

@frjo, setting up the this repo for the first time, while I know that the setup and everything is very similar to hypha.

When I run the server locally it still gives me sites configured for Open Technology Fund(E.g. Homepage etc.)

Is there some configuration that we need to do to have the sites configured with reset interface locally?

Also, please let me know if there is a db dump that you have specific to this repo.

Or if there is a document on doing specific setup for reset.

frjo commented 4 years ago

@sks444 Reset has custom models for front page etc. Will get you a sanitised db dump from Reset test, that will be the quickest and easiest I think.

sks444 commented 4 years ago

Yes, that will do the work. Thank you.

danblah commented 4 years ago

@sks444 and @frjo, responding.

Yes, let us call it partner.

Yes, that's right.

I've modified below.


Partner Organization:

Investment:

Investment level is right and okay. This assumes we would easily be able to search/filter/find organisations by fields of the investments associated with them.

They should be replaced by choosing a Category Question.

They should be replaced by choosing a Category Question.

They should be replaced by choosing a Category Question.

frjo commented 4 years ago

@sks444 We are moving to have this feature built in to Hypha directly so moving issue too the Hypha repo.

frjo commented 4 years ago

Hmm, thought you could move issues to any repo you had access to but it looks like it limited to repos within the same organisation? Anyone knows how to move it to OTF/hypha repo?

Parbhat commented 4 years ago

@frjo Looks like we can not transfer issues to different organisation. Maybe we can keep the issue here and @sks444 can create PR in Hypha repo.

Parbhat commented 4 years ago

@frjo Creating this feature on Hypha repo means we will need style changes both for OTF and Reset. As these pages will be on public site and both OTF and Reset have different design.

frjo commented 4 years ago

@Parbhat Yes, Reset will as a minimum need their own template for this page.

frjo commented 3 years ago

@danblah This can be tested at https://test.opentech.fund/

See https://github.com/OpenTechFund/hypha/pull/2126 for how and what to test.

danblah commented 3 years ago

Modifications to what we have right now... Index page

Partner page

Things to add... Investment page

Partners page

Index page

Even more general...

sks444 commented 3 years ago

@danblah, thanks for the feedback. Will improve the current functionalities and the new ones.

Regarding category questions: Can we have a fixed number fields with fixed number of choices(see image)?

This feature works very differently than application forms. We should consider these points before building the category questions feature for this:

I think Luminate might be doing something similar and have the following number of fields with constant choices:

Screenshot 2020-09-16 at 11 40 19 AM

Dan, considering the above points, can you please confirm once:

scott-psycle commented 3 years ago

@danblah Do you need our assistance implementing the Reset front end?

danblah commented 3 years ago

heya @sks444!

How do we reconcile the need for a barebones mvp short-term and these needs mid-term? Is there a small function you can build to generate the fixed fields and choices from existing Category Questions now? Is there a hook for changes to Category Questions you can use to regenerate these fields whenever Category Questions change? Good/bad ideas?

Can you lay out a new issue for meeting the mid-term needs above as a more permanent solution?

Thanks again!

danblah commented 3 years ago

@scott-psycle, as @sks444 gets all the values spitting out, it'd be great for you to take on refining the look/feel and extending any of the design templates as needed. @sks444 and @frjo, y'all sync up with @scott-psycle to figure out where the right hand-off is?

sks444 commented 3 years ago

Thanks for the inputs @danblah, that is very helpful!

I had a discussion with @Parbhat. We have a plan and we are going to try out the implementation for dynamically creating category questions and their choice selection for this feature.

sks444 commented 3 years ago

Hi @scott-psycle,

Yes, once we are done with implementing this in hypha. We shall discuss on having this feature in reset and making the design better. We will let you know once this is done. Thanks. cc: @frjo

sks444 commented 3 years ago

Hi all, a good amount of work has been done on this. Here are the updates to get further feedback:

I will quote the @danblah comments on requested changes and make a tick mark if that is done. And will also add a small description on the solution provided if needed.

Modifications to what we have right now... Index page

Partner page

Things to add... Investment page

Partners page

Index page

Even more general...

Adding Category Questions To Investments

sks444 commented 3 years ago

@danblah this is deployed to test with the new updates.

danblah commented 3 years ago

@sks444, just testing things out... super impressed.

we're nearly to mvp!

Im getting some 500 errors when i try to edit existing records btw, see https://test-apply.opentech.fund/admin/partner/investment/edit/4/. I think there's some bug where multiple partners are being created as well, see https://test.opentech.fund/results/new-partner-page/partner/ and https://test.opentech.fund/results/new-partner-page/partner/, which i think should be the same partner?

On the portfolio page (see https://test.opentech.fund/about/portfolio/)

On partner page (see https://test.opentech.fund/results/new-partner-page/test-partner-a/)

That will get us to a production MVP for styling on reset.tech I think.

I also need a page that outputs Partner logo and name in a responsive grid, default to only showing those that are active, with the ability to filter for all or only inactive.

I also need the ability to add a Partners block on our front page that defaults to showing the most recently invested in partners (but also allows us to promote specific ones to the front).

sks444 commented 3 years ago

Hi @danblah,

Can you do a quick testing on this again?

Include the category questions, if they are set when you expand the investment detail

As per the old requirements we automatically create a column in the investments table for category questions that gets added for investments.

Adding Category Questions To Investments

  • Visit Apply-> Category Questions in admin. And create category questions that you need for investments.
  • Settings-> Investment Category Settings: Select the category questions that you want to use for investments. Make sure to add it for both the sites(I.e. apply and public, There is an option to select site while editing/creating settings in the top-right.)
  • Investments-> Add Investment: Now the form will also have the category questions that you added. Select the valid choices and create investment.
  • Visit the investment table and there you would see the category questions is automatically added to the columns.

I have already added some test data for this while testing..we can ignore the Test Partner 1 listed at https://test-apply.opentech.fund/admin/pages/926/, I believe that is a cached page which doesn't exist in the db..we can fix that whenever we redeploy https://github.com/HyphaApp/hypha/pull/2126.

Let us try to get this merged in this iteration..as it's getting harder to keep track of things with this big changes of a stale pr.

danblah commented 3 years ago

Merge it!

sks444 commented 3 years ago

Thanks @danblah.

@frjo can we redeploy https://github.com/HyphaApp/hypha/pull/2126 to test..but first we would have to make sure that no data from the old migration exist..I guess we may need to follow steps from https://github.com/HyphaApp/hypha/pull/2126#issuecomment-803902944 in order to delete Test partner 1.

Once that is deployed..I can do a quick testing to make sure that the migrations are not causing any issues..then we can go-ahead and merge this.

frjo commented 3 years ago

@sks444 Now redeployed to https://test-apply.hypha.app/ (OBS! the Hypha test site, cloned the DB from OTF test so same accounts etc.)

sks444 commented 3 years ago

Thanks @frjo.

It seems to be working fine.

We can merge this in and create new issue for the features that remain:

from https://github.com/ResetNetwork/apply-app/issues/79#issuecomment-790857792 :

  • [ ] List investments table under a partner page

  • [ ] I also need a page that outputs Partner logo and name in a responsive grid, default to only showing those that are active, with the ability to filter for all or only inactive.

  • [ ] I also need the ability to add a Partners block on our front page that defaults to showing the most recently invested in partners (but also allows us to promote specific ones to the front).

frjo commented 3 years ago

The PR https://github.com/HyphaApp/hypha/pull/2126 is now merged.

Added three new issues, first to Hypha and the last two to Reset as they seem Reset specific.