Hi devs,
I don't think there is currently a process that is in place to handle
pull requests and I have the feeling that the way there are handled
today is a bit random.
There are usually comments sent out on each pull request but sometimes
it seems that some pull requests are going in sleep mode and it's not
clear who is in charge.
I would like to suggest that a process is put in place where it's
clear who is responsible for a pull request and a status is given to
the contributors that propose that pull request.
Something like:
Assigned developer: XXXX
Status:
New -> new pull request, not yet assigned
Assigned -> assigned to a developer, he is in charge of reviewing the
pull request and ask for modifications or accept it. The developer can
auto assign it to himself. If nobody does, we need to decide how they
will be taken into account.
ModificationsRequired -> for now rejected with comments. Contributor
needs to apply comments and then change back to Assigned for further
evaluation
VoteRequired -> there are no more comments, but a vote is required as
the changes to XWiki core are important
WaitingFinalAuthorization -> optional step for complex patches where
a additional authorization would be required (need to define who would
be the persons that give the authorization)
WaitingApplication -> there are no more comments and no changes or
vote required. The pull request can be applied and is waiting for a
developer to apply it
Abandoned -> contributors is abandoning the pull request (cannot do
the changes, no more time, etc..)
Rejected -> pull request is rejected (quality not enough, etc..)
Applied -> pull request is applied
What do you think ?
Ludovic
--
Ludovic Dubost
Founder and CEO
Blog: http://blog.ludovic.org/
XWiki: http://www.xwiki.com
Skype: ldubost GTalk: ldubost
Hello,
Right now class property hints are not supported right in the core of
XWiki (though it's been mentioned as a possible future item of the
roadmap for AWM, see http://markmail.org/message/ltuphlj7bnnso2yb ).
Once this will be implemented, it will rely on i18n message keys so
that hints are localizables (same strategy as for class property
pretty names)
I'd like to start the discussion and see if we can agree on a
convention that would be used in the future when this module is
implemented.
The reason for this is I'm starting to work on a LDAP admin UI, and I
find that their field pretty names are not self-explanatory. It's not
a matter of expression, it's just the fact that what they describe
needs more explanation than what the pretty name can offer). In this
light, I'm planning on displaying such longer hints in this admin
section.
Right now the format for a pretty name is :
Space.Class_property=Pretty name
The format for a hint could be something like :
Space.Class_property.xHint=Hint
This would need escaping rules I think, since I believe
"property.xHint" is a valid property name
Note that I use "xHint" and not "hint" to be consistent with the form
standards class names. This can be discussed though.
WDYT ? Do you have other ideas ?
Jerome.
--
Jérôme Velociter
Winesquare
http://www.winesquare.net/
Hi devs,
On http://dev.xwiki.org/xwiki/bin/view/Community/ApplicationDevelopmentBestPra… we list some app dev best practices. I’m proposing to add one more bullet point which is that apps bundle one or several (but at least one) Dashboard Gadget (technically a Macro) so that users can easily add a view of the app on the home page (in their dashboard).
For example:
* FAQ app already provides a {{faq}} gadget to list all entries for a given FAQ app
* Index app already provides a {{document}} gadget to list all documents matching some criteria
* Blog app doesn’t currently provide such a gadget (it provides a velocity macro but that’s not a gadget)
* Forum app should provide a {{forums}} gadget too to list all forums and a {{forum}} one to list entries from a given forum
* etc
WDYT?
Thanks
-Vincent
Hi devs,
Since statistics are disabled by default, I'm proposing to not bundle the
Statistics application by default. Admins who want to enable stats on their
would need to install it through the EM.
In addition, the Stats app's quality is not exactly perfect either and
performances are not that great so I think it makes sense to not promote it
too much either.
Last, since 5.3M2 the stats app is now visible in the Applications Panel
(for Admins), thus not bundling it by default seems even more needed now
IMO.
WDYT?
Thanks
-Vincent
Hi devs,
Here’s what I commented on https://github.com/xwiki/xwiki-platform/pull/307 :
“
Thanks Pascal. I've just noticed that we still bundle TinyMCE in XWiki's WAR. It was probably left for backward compatibility but since it's been like 6-7 years that we've dropped it, I believe we could vote about dropping it from our sources and if someone really needs it they'll still be able to manually install them in their wikis. I'll follow up on the devs list, thanks.
“
WDYT? Are you ok to remove it?
Thanks
-Vincent
Hi devs,
So this proposal appeared in some of my proposals but right now I created a
proper Proposal/Idea mail for it.
It's about having an 'Administer Page' section in the menus, similar to the
'Administer Wiki' and 'Administer Space' functionality.
The 'Administer Page' will be accessible to global admins, page creators
and users with (edit?/admin?) rights for the page.
>From what we currently have implemented it should contain the 'Rights
Editor' (?editor=rights) at page level.
In the future we could make 'Presentation', 'Page Elements' or 'Panel
Wizard' to be more granular and be set at Page level (have different panels
and style for just one page).
Some screenshots at
http://incubator.myxwiki.org/xwiki/bin/view/Improvements/PageAdministration
Thanks,
Caty
Hi,
We are using mixed naming when referring to a document/page, not only on
different pages, but also in the same context (Rename step for example).
There is also an issue referring to this problem
XWIKI-5401<http://jira.xwiki.org/jira/browse/XWIKI-5401>and we need to
find an answer in order to move forward consistency.
So the question is which version we prefer: "Page" or "Document" ?
I'm voting +1 for "Page".
"Page" is more used IMO, especially in the "Space"/"Page" context.
"Page" is more general than "Document" and better fitted for a platform that
encapsulates all kind of content (not just documents).
Please cast your vote.
Thanks,
Caty
Hi,
According to the javadoc in XWikiDocument:
/**
* The last user that has changed the document's content (ie not object, attachments). The Content author is only
* changed when the document content changes. Note that Content Author is used to check programming rights on a
* document and this is the reason we need to know the last author who's modified the content since programming
* rights depend on this.
*/
private DocumentReference contentAuthorReference;
This means that objectadd or objectremove actions shouldn't change the content author as they do now.
I'm proposing that we fix this.
Do you see any issue?
Thanks
-Vincent
Hi devs,
In XWiki, if you send a POST request at the following URL
('MySpace.MyPage' is a document that doesn't exist at the moment)
/bin/save/MySpace/MyPage
with the following HTTP parameters:
* template=MySpace.MyTemplate
* XWiki.XWikiRights_0_users=XWiki.Me
* <others parameters>
with 'MySpace.MyTemplate' a template document and 'XWiki.Me' a user of
the wiki.
2 cases:
1. If 'MySpace.MyTemplate' contains a 'XWiki.XWikiRights' object, then
the resulting document 'MySpace.MyPage' will have this object with its
property 'users' initialize with the value 'XWiki.Me' (see HTTP
parameters)
2. If 'MySpace.MyTemplate' does not contain a 'XWiki.XWikiRights'
object, the resulting document 'MySpace.MyPage' will not contain it
either
### PROPOSAL
Create automatically the objects if they don't exist in the template
document.
###
To make it possible, it seems that we would need to refactor the method
'readObjectsFromForm(EditForm, XWikiContext)' in the
'com.xpn.xwiki.doc.XWikiDocument' class. And probably modify also the
'getObject(String)' from 'com.xpn.xwiki.web.EditForm' class.
2 things to take care:
* It seems there is no unit test for these methods.
* May this proposal be a security problem?
WDYT?
--
Jean