Plots2: Change texts for "settings" page

Created on 27 Nov 2019  Â·  27Comments  Â·  Source: publiclab/plots2

Problem

We need to make some text adjustments on this page, https://publiclab.org/settings :

image

To work on :

@steviepubliclab has developed some new text for it! Thanks!

Email Notifications

I want to receive an email

- Whenever there are new posts on topics I follow
- In a daily digest
- In a weekly digest

I want to be notified by email for:

- Comments on my posts
- Likes on my posts
- Comments on all posts I’ve commented on

Browser Notifications

I want to receive browser notifications:

- When I’m mentioned
- When someone likes my work

Let's make these text changes on this page: https://github.com/publiclab/plots2/commits/main/app/views/users/settings.html.erb

_Originally posted by @jywarren in https://github.com/publiclab/plots2/issues/6867#issuecomment-559195780_

HTML assigned help wanted

All 27 comments

I would like to work on this

Please do, we'd love your help!

On Thu, Nov 28, 2019, 9:55 AM kush aggarwal notifications@github.com
wrote:

I would like to work on this

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/publiclab/plots2/issues/6871?email_source=notifications&email_token=AAAF6JYHK3335YFTJAWXMLLQV7LS7A5CNFSM4JSM5CD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFM2MPQ#issuecomment-559523390,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAF6JZYV5ZSLZ6MA3WQMDDQV7LS7ANCNFSM4JSM5CDQ
.

@pydevsg this issue is already taken by @kushaggarwal so we can't keep it in GCI.
First come first serve is followed in OS.
@kushaggarwal in case you want any help to solve this issue please feel free to ping me or any other mentor.
Best of luck @kushaggarwal
Regards
Sidharth

Okay Thanks @SidharthBansal
Will surely contact in case of any problem

Hi I was trying to set up the project inorder to work for the problem above but not able to set it up it says the error as shown image
Can you please help
Screenshot 2019-12-02 at 7 37 40 PM

Have you updated your bundle and done bundle install?

Yes I have executed the bundle update command but it throws the below error first and then the one in above image
Screenshot 2019-12-02 at 10 49 02 PM

@kushaggarwal

I just tried it and the installation worked for me on Ubuntu 18.04.3 LTS.

It appears you are running Mac OS, which might have some platform-specific problems. I found this StackOverflow article which might work for you.

gem uninstall libv8
gem install therubyracer -v '0.11.3'
gem install libv8 -v '3.11.8.13' -- --with-system-v8

Hi all, @kushaggarwal, were u able to fix your issue. Thanks @chen-robert @SidharthBansal :+1:

Yes I am on it will update you soon about the same

Thanks @chen-robert for your help
Will go through the article I hope this will work

Hi @chen-robert
I went through the stackoverflow article you mentioned in the above message but was not able to resolve the error, the sam error still persists so can you please help
Also can yo u please help what are the previously installed dependencies that should be installed in the system if any as I might be missing any leading to this error
Thanks

Hey @chen-robert ,
You could try this command gem update --system if that fails try this sudo apt-get install ruby-dev zlib1g-dev liblzma-dev

Hello guys,
Sorry for the delay was caught up with my university exams
Will soon make a pull request for the above issue
Thank You

Hello
In above mentioned layout the slider buttons need to be there for each sub option in the heading like I want to be notified by email

Hi @kushaggarwal, I believe the slider buttons are already in place...we need to change the text. Thanks

Okay Thank you will do so

@publiclab/reviewers Can I take up this task if it's still not done?

Hello
I am working on this one and will soon generate a pull request for the same
Thank you

Fine @kushaggarwal ,take your time to complete the task.Thanks

@jywarren Can I work on this issue?

Hello
I have already submitted a pull request regarding the same

@kushaggarwal then you should edit your pr and write the issue number their in fixes.

Okay will do so
Thank you

Hi folks, just noting that there was a very good start to this in https://github.com/publiclab/plots2/pull/7259 which could be build upon if someone else would like to take this up!

I would like to work on this!

Was this page helpful?
0 / 5 - 0 ratings

Related issues

first-timers[bot] picture first-timers[bot]  Â·  3Comments

first-timers[bot] picture first-timers[bot]  Â·  3Comments

ebarry picture ebarry  Â·  3Comments

keshavsethi picture keshavsethi  Â·  3Comments

grvsachdeva picture grvsachdeva  Â·  3Comments