NinjaReader / AndroidNinjaReader

A Fun way to read more and stop piling up articles
0 stars 2 forks source link

Added the mocks #7

Closed ccoria closed 9 years ago

ccoria commented 9 years ago

@alonrz and @mvinceable,

I Added the mocks to this project. I think perhaps we will need to create an action bar to make it look better.. I have also added the models, icons and updated the styles..

I will work on the stats page tomorrow.

Just to make sure we are in the same page:

Is that right? Am I missing something? Do you guys agree?

There is this action bar styling that we may need to do. I think whoever is free first could take a look on that.

Thanks, Cassiano

mvinceable commented 9 years ago

That sounds good to me.

Thanks, Vince

Sent from my iPhone

On Feb 25, 2015, at 12:04 AM, Cassiano Coria Neto notifications@github.com wrote:

@NinjaReader/owners,

I Added the mocks to this project. I think perhaps we will need to create an action bar to make it look better.. I have also added the models, icons and updated the styles..

I will work on the stats page tomorrow.

Just to make sure we are in the same page:

Vince will bind the mocks to his backend Alon will work on the intents to add articles and eventually on alerts Cassiano will do the stats page Is that right? Am I missing something? Do you guys agree?

There is this action bar styling that we may need to do. I think whoever is free first could take a look on that.

Thanks, Cassiano

— Reply to this email directly or view it on GitHub.

mvinceable commented 9 years ago

Hey Guys,

There were some bugs with parse login that I fixed now. We should only see the reading list when there's a proper current user. I also added a ParseObject for Article. Please pull for the latest as I've also restructured the files for better organization.

Thanks, Vince

On Wed, Feb 25, 2015 at 12:04 AM, Cassiano Coria Neto < notifications@github.com> wrote:

@NinjaReader/owners https://github.com/orgs/NinjaReader/teams/owners,

I Added the mocks to this project. I think perhaps we will need to create an action bar to make it look better.. I have also added the models, icons and updated the styles..

I will work on the stats page tomorrow.

Just to make sure we are in the same page:

  • Vince will bind the mocks to his backend
  • Alon will work on the intents to add articles and eventually on alerts
  • Cassiano will do the stats page

Is that right? Am I missing something? Do you guys agree?

There is this action bar styling that we may need to do. I think whoever is free first could take a look on that.

Thanks, Cassiano

— Reply to this email directly or view it on GitHub https://github.com/NinjaReader/AndroidNinjaReader/issues/7.

mvinceable commented 9 years ago

Hi,

In the ReadingListActivity I added an example for getting article info from Readability and storing values we care about into Parse. I'm seeing live data in Parse now.

Thanks, Vince

On Wed, Feb 25, 2015 at 2:47 AM, Vince vmagistrado@gmail.com wrote:

Hey Guys,

There were some bugs with parse login that I fixed now. We should only see the reading list when there's a proper current user. I also added a ParseObject for Article. Please pull for the latest as I've also restructured the files for better organization.

Thanks, Vince

On Wed, Feb 25, 2015 at 12:04 AM, Cassiano Coria Neto < notifications@github.com> wrote:

@NinjaReader/owners https://github.com/orgs/NinjaReader/teams/owners,

I Added the mocks to this project. I think perhaps we will need to create an action bar to make it look better.. I have also added the models, icons and updated the styles..

I will work on the stats page tomorrow.

Just to make sure we are in the same page:

  • Vince will bind the mocks to his backend
  • Alon will work on the intents to add articles and eventually on alerts
  • Cassiano will do the stats page

Is that right? Am I missing something? Do you guys agree?

There is this action bar styling that we may need to do. I think whoever is free first could take a look on that.

Thanks, Cassiano

— Reply to this email directly or view it on GitHub https://github.com/NinjaReader/AndroidNinjaReader/issues/7.

mvinceable commented 9 years ago

Please pull the latest changes before working further as I had to resolve some naming conflicts.

alonrz commented 9 years ago

Hi guys, great work. I will add the action bar too as I need to add a button to it anyways.

On Wed, Feb 25, 2015, 03:31 mvinceable notifications@github.com wrote:

Please pull the latest changes before working further as I had to resolve some naming conflicts.

— Reply to this email directly or view it on GitHub https://github.com/NinjaReader/AndroidNinjaReader/issues/7#issuecomment-75945601 .

alonrz commented 9 years ago

Hey Vince, is the login big you fixed related to what I sent you?

On Wed, Feb 25, 2015, 04:41 Alon RZ alon.zilberman@gmail.com wrote:

Hi guys, great work. I will add the action bar too as I need to add a button to it anyways.

On Wed, Feb 25, 2015, 03:31 mvinceable notifications@github.com wrote:

Please pull the latest changes before working further as I had to resolve some naming conflicts.

— Reply to this email directly or view it on GitHub https://github.com/NinjaReader/AndroidNinjaReader/issues/7#issuecomment-75945601 .

ccoria commented 9 years ago

Great additions, thanks guys.

I also added the fragment where we can load an article to be read (ArticleFragment.java). It is there but it is not being loaded.

Hey @mvinceable, are you planning to remove this example from our namespace? I can do it but the chance of having conflicts is very high..

On the stats, I researched a little and I will user this library to the graphs: https://github.com/HackPlan/AndroidCharts

Would you guys be able to meet tomorrow from 2PM to 3PM to talk about the last details?

Thanks, Cassiano

alonrz commented 9 years ago

im free. can you create an invite in yahoo cal?

On Wed Feb 25 2015 at 11:30:59 AM Cassiano Coria Neto < notifications@github.com> wrote:

Great additions, thanks guys.

I also added the fragment where we can load an article to be read (ArticleFragment.java). It is there but it is not being loaded.

Hey @mvinceable https://github.com/mvinceable, are you planning to remove this example from our namespace? I can do it but the chance of having conflicts is very high..

On the stats, I researched a little and I will user this library to the graphs: https://github.com/HackPlan/AndroidCharts

Would you guys be able to meet tomorrow from 2PM to 3PM to talk about the last details?

Thanks, Cassiano

— Reply to this email directly or view it on GitHub https://github.com/NinjaReader/AndroidNinjaReader/issues/7#issuecomment-76037075 .

ccoria commented 9 years ago

Sure, done

mvinceable commented 9 years ago

Guys, anyone should be free to make any changes to this repo.

@ccoria I didn't do anything with ArticleFragment. Perhaps if it contained live data it might be helpful.

alonrz commented 9 years ago

I wonder if we should invest time with real data at this point, or just static screens. When I try, there are all these bugs that float up. So I didnt do the saving part properly yet. Just to demo there is the share and add article dialog from the app. I set the structure for future dev but when I tried there were a bunch of crashes and scenarios that needs attention. What do you guys think?

ccoria commented 9 years ago

Guys,

I created this profile/stats page. What do you think?

I still have to integrate with this project.. Sorry the Graphs API was kind of hard to customize. And I had to change to the hellocharts because the other one was not being nice with me.

I think that we have just to tell the story.. if we have it working, ok, but if not, lets try to tell a good story.. =)

Thanks, Cassiano screen shot 2015-02-26 at 12 31 20 am

ccoria commented 9 years ago

@mvinceable I agree and I would change the namespace but since you were working on the classes itself I was concerned about conflicts. But sure, we are free do to whatever.. =)

mvinceable commented 9 years ago

With the latest changes I checked in, we have a working prototype that is integrated with Parse that can

1) List articles user has saved 2) Mark articles as read via long press 3) Show word count for all articles and lead image if available

I also went in and fixed all the bugs I found. Should be pretty stable now.

@alonrz I was able to merge with your changes also @ccoria that chart looks nice. I think we can go even more granular and add word counts?