-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
consolidate branches #143
consolidate branches #143
Conversation
line-o
commented
Jan 24, 2017
•
edited
Loading
edited
- adds all commits in master that were missing from develop
- bumps version to 2.3.0
It's convenient to be able to click on the label of an otherwise very small checkbox control. I'm happy to apply this elsewhere; I think there are many similar controls whose labels could be made clickable.
[feature] Make "remember me" label clickable
* master: [feature] Make "remember me" label clickable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@line-o These changes look good. I'm curious: is there a reason you're bumping to 2.3.1 and not 2.3.0? Also, I have some notes to add to repo.xml describing the additions in this release. I could submit these as part of this PR or as a separate one - what would you prefer?
The repo.xml addition would be as follows (replacing 2.3.1 with 2.3.0 if you agree that 2.3.0 is better, since there was no 2.3.0 release to my knowledge): <change version="2.3.1">
<ul xmlns="http://www.w3.org/1999/xhtml">
<li>New adaptive and JSON output options for query results</li>
<li>New checkbox to toggle highlighting of index hits</li>
<li>Fix function doc lookup</li>
<li>Various small bug fixes</li>
</ul>
</change> Again, I'm happy to submit this in this or in a separate PR. |
@joewiz 2.3.0 is the current one in the package repository. |
👍 |
@line-o Ah, I see. Something strange is going on though, since I've never seen 2.3.0 via the package manager pane in Dashboard. Indeed, even running the latest nightly, the package repository still shows 2.2.0 to me: (This is showing what's available from the repo, not what's installed on my machine.) The package manager's request that returns this entry is: http://localhost:8080/exist/apps/dashboard/plugins/packageManager/packages/?format=manager&type=remote. I wonder if this request isn't properly sending the version parameter that the package manager relies on to determine compatibility. I've fixed some aspects of the public-repo and dashboard, but I don't think I've fixed this request. I do know that the package repo (at least the one on exist-db.org) has some bugs that I've fixed in source but aren't running on exist-db.org. I'll look into this package manager request bug later this morning. Update: There was no bug in the package manager; as noted below, the problem was in the contents of the public repo. |
Wait, take a look at http://demo.exist-db.org/exist/apps/public-repo/packages/eXide.html?eXist-db-min-version=3.0.0 - 2.3.0 is not listed. Also, the 2.3.0 version number was not in source - as the diff of this PR shows. So if 2.3.0 was in the public-repo, I don't think the listing of apps was ever refreshed (a manual step separate from uploading the .xar), which would've updated the listing on this URL. So no one could've installed it. In light of this, I think 2.3.0 was never "released" - so by that logic, should the version number should be switched to 2.3.0? |
You are right, v2.3.0 it is then. |
- Moved the listing of 2.0.8's "new features" down to the main section, edited slightly to be regular documentation rather than "new features" - Added 2.3.0's new features
This is now in build.properties
Note: When 2.3.0 is released, please push this update to http://exist-db.org/exist/apps/eXide. It's running a veeery old version of eXide now.