Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Max Woghiren]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/
---

(Updated Oct. 21, 2013, 6:04 p.m.)


Review request for cordova.


Repository: cordova-site


Description
---

A blog post outlining the basics of the Cordova plugin registry.


Diffs (updated)
-

  /README.md 1533594 
  /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 

Diff: https://reviews.apache.org/r/14757/diff/


Testing
---


Thanks,

Max Woghiren

Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Max Woghiren]

> On Oct. 21, 2013, 3:13 p.m., Michal Mocny wrote:
> > /www/_posts/2013-10-21-cordova-registry.md, line 11
> > 
> >
> > s/what plugins/which plugins/

I believe "what" is correct here.  "Which" is for when you're picking from a 
set of items; "what" is used when the options to choose from aren't known.


- Max


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/#review27246
---


On Oct. 18, 2013, 7:59 p.m., Max Woghiren wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14757/
> ---
> 
> (Updated Oct. 18, 2013, 7:59 p.m.)
> 
> 
> Review request for cordova.
> 
> 
> Repository: cordova-site
> 
> 
> Description
> ---
> 
> A blog post outlining the basics of the Cordova plugin registry.
> 
> 
> Diffs
> -
> 
>   /README.md 1533594 
>   /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/14757/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Max Woghiren
> 
>

Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Max Woghiren]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/#review27248
---



/www/_posts/2013-10-21-cordova-registry.md


Merged this line with the previous line.

I actually think I want to push back on aiming the post at app developers.  
If I were to change the audience to app developers, I'd remove the information 
on publishing plugins, since they don't care about that.  A neutral audience 
makes sense.

Reworded the sentence; good point.



/www/_posts/2013-10-21-cordova-registry.md


No harm in having it there.  It feels wrong to say "first, you need to 
create an account".  Most of the time, that's false. :)

I changed the wording a bit, though, to make it a little less jarring.



/www/_posts/2013-10-21-cordova-registry.md


I've removed the unpublish mention.  I don't know that I want to add too 
many details, but I changed it to "such as search for plugins by keyword".  
There's no preferred method of plugin searching.



/www/_posts/2013-10-21-cordova-registry.md


Changed "access" to "use".


- Max Woghiren


On Oct. 18, 2013, 7:59 p.m., Max Woghiren wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14757/
> ---
> 
> (Updated Oct. 18, 2013, 7:59 p.m.)
> 
> 
> Review request for cordova.
> 
> 
> Repository: cordova-site
> 
> 
> Description
> ---
> 
> A blog post outlining the basics of the Cordova plugin registry.
> 
> 
> Diffs
> -
> 
>   /README.md 1533594 
>   /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/14757/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Max Woghiren
> 
>

Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Michal Mocny]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/#review27246
---



/www/_posts/2013-10-21-cordova-registry.md


s/what plugins/which plugins/



/www/_posts/2013-10-21-cordova-registry.md


Consider merging these two lines, and imho consider the reader an app 
developer instead of being abstract.

"Using the Cordova CLI, you can add plugins to your workspace with a single 
easy command:"

I wouldn't mention messy details (it badmouths the plugman flow, and sets 
up for a reply to say "hey I did blah with CLI and now I have messy details, 
you liar")



/www/_posts/2013-10-21-cordova-registry.md


"if you dont already have one" is implied.



/www/_posts/2013-10-21-cordova-registry.md


Not sure we need to call out unpublish, but search command is interesting.  
Is it prefered to use plugman to search, or to visit plugins.cordova.io?  Is 
the list the same?  Maybe expand on that.



/www/_posts/2013-10-21-cordova-registry.md


"app developers" -> "you", "their" -> "your", imho

"access" -> "use"


- Michal Mocny


On Oct. 18, 2013, 7:59 p.m., Max Woghiren wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14757/
> ---
> 
> (Updated Oct. 18, 2013, 7:59 p.m.)
> 
> 
> Review request for cordova.
> 
> 
> Repository: cordova-site
> 
> 
> Description
> ---
> 
> A blog post outlining the basics of the Cordova plugin registry.
> 
> 
> Diffs
> -
> 
>   /README.md 1533594 
>   /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/14757/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Max Woghiren
> 
>

Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Steven Gill]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/#review27202
---

Ship it!


Looks Great!

- Steven Gill


On Oct. 18, 2013, 7:59 p.m., Max Woghiren wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14757/
> ---
> 
> (Updated Oct. 18, 2013, 7:59 p.m.)
> 
> 
> Review request for cordova.
> 
> 
> Repository: cordova-site
> 
> 
> Description
> ---
> 
> A blog post outlining the basics of the Cordova plugin registry.
> 
> 
> Diffs
> -
> 
>   /README.md 1533594 
>   /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/14757/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Max Woghiren
> 
>

Re: Review Request 14757: Cordova Plugin Registry Blog Post

[Anis Kadri]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/#review27201
---

Ship it!


Ship It!

- Anis Kadri


On Oct. 18, 2013, 7:59 p.m., Max Woghiren wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14757/
> ---
> 
> (Updated Oct. 18, 2013, 7:59 p.m.)
> 
> 
> Review request for cordova.
> 
> 
> Repository: cordova-site
> 
> 
> Description
> ---
> 
> A blog post outlining the basics of the Cordova plugin registry.
> 
> 
> Diffs
> -
> 
>   /README.md 1533594 
>   /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 
> 
> Diff: https://reviews.apache.org/r/14757/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Max Woghiren
> 
>

Review Request 14757: Cordova Plugin Registry Blog Post

[Max Woghiren]

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14757/
---

Review request for cordova.


Repository: cordova-site


Description
---

A blog post outlining the basics of the Cordova plugin registry.


Diffs
-

  /README.md 1533594 
  /www/_posts/2013-10-21-cordova-registry.md PRE-CREATION 

Diff: https://reviews.apache.org/r/14757/diff/


Testing
---


Thanks,

Max Woghiren