Skip to content
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

Merged
merged 11 commits into from
Feb 5, 2017
Merged

consolidate branches #143

merged 11 commits into from
Feb 5, 2017

Conversation

line-o
Copy link
Member

@line-o line-o commented Jan 24, 2017

  • adds all commits in master that were missing from develop
  • bumps version to 2.3.0

joewiz and others added 4 commits July 18, 2016 13:53
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
Copy link
Member

@joewiz joewiz left a 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?

@joewiz
Copy link
Member

joewiz commented Jan 24, 2017

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.

@line-o
Copy link
Member Author

line-o commented Jan 24, 2017

@joewiz 2.3.0 is the current one in the package repository.
The additions to repo.xml should be part of this one. Good that you noted it, as I did not.

@line-o
Copy link
Member Author

line-o commented Jan 25, 2017

👍

@joewiz
Copy link
Member

joewiz commented Jan 25, 2017

@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:

screen shot 2017-01-25 at 7 45 22 am

(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.

@joewiz
Copy link
Member

joewiz commented Jan 25, 2017

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?

@line-o
Copy link
Member Author

line-o commented Jan 25, 2017

You are right, v2.3.0 it is then.

line-o and others added 5 commits January 25, 2017 14:34
- 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.
@wolfgangmm wolfgangmm merged commit add22b9 into eXist-db:develop Feb 5, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants