Closed Bug 1141971 Opened 9 years ago Closed 9 years ago

[Privacy Controls] fix remaining words

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect)

Other
Gonk (Firefox OS)
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gasolin, Assigned: yuwen41200.cs03, Mentored)

References

Details

(Whiteboard: good first bug)

Attachments

(1 file)

Some wording indicates 'privacy panel' should all be 'privacy controls'

http://transvision.mozfr.org/?recherche=privacy+panel&repo=gaia&sourcelocale=en-US&locale=en-US&search_type=strings
YuWen would like to fix this issue. It's a pretty simple bug so the main effort would be to familiar with the contribution process.

Please take a look on
https://leanpub.com/gaiafromabove/read#leanpub-auto-chapter-2---get-in-touch-with-gaia

Which supposedly provide the integrated guide from getting source code, how code is organized, and where to modify the string for localization (hint 2.7.3)

If anything not that clear to you, feel free to ask question on IRC moznet > #mozilla-taiwan or just ask in this thread.
Assignee: nobody → yuwen41200.cs03
Pull Request #28877
https://github.com/mozilla-b2g/gaia/pull/28877
Thanks for providing your first pull request(PR)!

I've check the PR and find some nits that can be improved. Please check them on github. Once you done the change, please squish your commits into single commit so we can have more clear commit log while the PR is merged.

(Here is the reference about how to squich commit http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html)

To properly get the commit reviewed, please use 'Add an attachment' and paste your Pull Request on file field. Then set review flag to Marta (marta@sec.t-labs.tu-berlin.de), who is the module owner of Privacy Controls.
Flags: needinfo?(yuwen41200.cs03)
(In reply to Fred Lin [:gasolin] from comment #3)
> I've check the PR and find some nits that can be improved. Please check them
> on github. Once you done the change, please squish your commits into single
> commit so we can have more clear commit log while the PR is merged.
Since the commit history contains commits from upstream, and the PR is undergoing tests, I wonder if it is appropriate to squish all commits together now.
Flags: needinfo?(yuwen41200.cs03)
Attachment #8577868 - Flags: review?(marta)
Ooh I did not make the instruction clear. You should not use `git pull` to update your pull request. The best way to keep your commits above recent Gaia changes is use command:

```
$ git remote update
$ git rebase upstream/master`
```

Sorry I've not mention that. Could you open another pull request to provide your patch with single commit?
Flags: needinfo?(yuwen41200.cs03)
Comment on attachment 8577868 [details] [review]
[gaia] yuwen41200:master > mozilla-b2g:master

According to above comment, its better to wait for a clean PR for review.
Attachment #8577868 - Flags: review?(marta)
Thanks! I have pushed the squashed commit.
Now the PR should be correct.
https://github.com/mozilla-b2g/gaia/pull/28877
Flags: needinfo?(yuwen41200.cs03)
Thanks for your effort. Looks good to me.
It's time to set the flag for review!
Attachment #8577868 - Flags: review?(marta)
Attachment #8577868 - Flags: review?(marta) → review+
Yu-wen, Congrets for your first review+!

merged to master https://github.com/mozilla-b2g/gaia/commit/f66682c9e9d291c41f349ea4481830c5a8935329
thanks!
Status: NEW → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.