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

Sync upstream 2.6 latest #4

Open
wants to merge 86 commits into
base: dia
Choose a base branch
from
Open

Conversation

catalyst-joe-guo
Copy link

Fixes #

Proposed fixes:

Features:

  • includes tests covering changes
  • includes updated documentation
  • includes user-visible changes
  • includes API changes
  • includes bugfix for possible backport

Please [X] all the boxes above that apply

amercader and others added 30 commits November 29, 2016 13:07
Session, that used for querying config varibales diring startup
was not finished and one idle connection appeared in postgres.

Now session committed in the end of environment setup.
`datastore_create` directly updates database and solr index and
this reduces possibility of conflicts inside simultaneous calls
Small change in datastore plugin, that generates
fully-qualified url during datastore creation
The problem was that dashboard_mark_activities_old method was not changed to utcnow() format, so when datasets were updated, because of the time difference, user have seen them before he actually entered the page.

I've adde utcnow() format for this method and now the time works fine. Also added untnow() to Dashboard model, so the time can be saved correctly.
Now documentation contains correct default value - False.
Also this option mentioned in writhing extension guide
Added few additional allowed tags to `bleach.clean`
function and changed sanitization sequence so that
markdown applied first and only after that result cleaned
Aleksandar Jovanov and others added 24 commits July 5, 2017 11:47
the code below
```
    if not authorized1 and authorized2:
        return {'success': False, 'msg': _('User %s not authorized to edit these packages') % user}
    else:
        return {'success': True}
```
didn't work when authorized1 == authorized2 == False which presented that user cannot access to both packages(datasets).    
It would be : `if not False and False` -> `if True and False` -> `if Fasle`, then return `'success' : True`
Dataset counts were not displayed at all for group items because
the template checked `packages` rather than `package_count`.
When ckan.i18n_directory was used we were passing the wrong keyword
(`config`) instead of `pylons_config`
…-upstream-2.6-latest

for get_reset_link, upstream already has a fix, so our own fix is not
necessary, removed.
@catalyst-joe-guo catalyst-joe-guo changed the base branch from master to dia August 3, 2017 05:13
@catalyst-joe-guo
Copy link
Author

Hi Karen:
I just park this PR here, you can merge it when you think it's ready.

@dythya
Copy link

dythya commented Jan 9, 2025

closing this was a while ago

@dythya
Copy link

dythya commented Jan 9, 2025

actually can't close. :-)

@dythya dythya removed their assignment Jan 9, 2025
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.