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

Add free label to free section of trial course in My Course Activity #164

Closed
aggarwalpulkit596 opened this issue May 15, 2019 · 29 comments · Fixed by #187 or #211
Closed

Add free label to free section of trial course in My Course Activity #164

aggarwalpulkit596 opened this issue May 15, 2019 · 29 comments · Fixed by #187 or #211

Comments

@aggarwalpulkit596
Copy link
Contributor

No description provided.

@aggarwalpulkit596
Copy link
Contributor Author

Go ahead

@shubhamofbce
Copy link
Contributor

I would like to solve this issue

@saumya1singh
Copy link

But free label is already marked here.

image

Am I right ? Please guide .

@aggarwalpulkit596
Copy link
Contributor Author

Add it to course individual sections

@luckyman20
Copy link

@SaumyaSingh1 You have to mark the sections of course content that are free, I guess.

Also, sorry, I won't be able to work on this issue anymore.

@saumya1singh
Copy link

Free label in few sections , is it something like this ...
image

@aggarwalpulkit596
Copy link
Contributor Author

Yes exactly like this

@saumya1singh
Copy link

so, upto how many sections should we mark free label ? And are we supposed to do this in all the courses ?

@aggarwalpulkit596
Copy link
Contributor Author

For all the sections which are not premium and obviously yes for all the courses

@saumya1singh
Copy link

okay :)

@saumya1singh
Copy link

@aggarwalpulkit596 I have added the free label . looks like this ..
image

But right now the label is for all sections . Need sometime to make it confined to the sections which are not premium :)

@aggarwalpulkit596
Copy link
Contributor Author

Does it good to you ? I don't think so make design similar to website

@saumya1singh
Copy link

Okay i will make design similar to the website .

@shubhamofbce
Copy link
Contributor

@aggarwalpulkit596 From where can we know that which courses are free and which are not??

@shubhamofbce
Copy link
Contributor

gradient in cb free
I am also at this stage and now i just want to know that from where i can get the data that which courses are free and which are not????

@shubhamofbce
Copy link
Contributor

I did it.
Is it alright or @aggarwalpulkit596 you need more modificitaions??
Its working fine for all the courses.

free working everywhere

@shubhamofbce
Copy link
Contributor

Its all set now looking exactly like the website can i send a pr now??? @aggarwalpulkit596

all set now

@aggarwalpulkit596
Copy link
Contributor Author

If @SaumyaSingh1 is okay with this you can send a PR @shubhamofbce

@shubhamofbce
Copy link
Contributor

@aggarwalpulkit596 please review

@aggarwalpulkit596
Copy link
Contributor Author

@shubhamofbce i told you @SaumyaSingh1 asked it first if she doesn't make a pr by tomorrow I'll your pr

@shubhamofbce
Copy link
Contributor

But i asked to work on this first , You can see the comments from the begining. I asked two days ago she commented today.

@aggarwalpulkit596
Copy link
Contributor Author

@SaumyaSingh1 i guess it's okay then

@saumya1singh
Copy link

@aggarwalpulkit596 Because of my exams i was not able to complete the task . But I have done half of the work .
Issue is still open ! are we still supposed to resolve same problem.
Thanks for the guidance @aggarwalpulkit596

@aggarwalpulkit596
Copy link
Contributor Author

@SaumyaSingh1 yeah you can still work on this follow the design which @shubhamofbce used but his method was incorrect so i had to revert that

@saumya1singh
Copy link

Okay

@shubhamofbce
Copy link
Contributor

Did @SaumyaSingh1 resolved the issue or is it still open??? @aggarwalpulkit596

@aggarwalpulkit596
Copy link
Contributor Author

It is still open

@shubhamofbce
Copy link
Contributor

@aggarwalpulkit596 I made a pr with proper logic this time do check it.

@shubhamofbce
Copy link
Contributor

@aggarwalpulkit596 #209 has no conflicts and got all tests passed too. please review for one last time.

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