Patricio Del Boca

icon indicating a website link https://pdelboca.me icon indicating an email [email protected]

icon company @okfn icon location The Internet icon location Information Systems Engineer. Proud member of @OpenDataCordoba and @ckan core developer. Working for a Fair, Sustainable and Open Future @okfn

Results 91 comments of Patricio Del Boca

Fixed Page View Tracking when CKAN is running on subpath

Hello @nisha-nec ! CKAN is a volunteer-based project. While we appreciate all contributions and Pull Requests we have limited time and we cannot provide a timeline for when (or if)...

Fix 1633 unexpected key error with primary key in schema and 1635 unexpected missing label error when ignoring header case

@pierrecamilleri we shoudn't be using `pytest.lazy_fixture` in our codebase since it has been deprecated: https://github.com/TvoroG/pytest-lazy-fixture/issues/16 Could we instead call `from pytest_lazyfixture import lazy_fixture` as suggested in https://github.com/TvoroG/pytest-lazy-fixture/issues/22#issuecomment-361976842. It is not...

Fix 1633 unexpected key error with primary key in schema and 1635 unexpected missing label error when ignoring header case

@pierrecamilleri wait, I will fix this in a new PR instead and then we can merge the fix into this two PRs.

Fix 1633 unexpected key error with primary key in schema and 1635 unexpected missing label error when ignoring header case

@pierrecamilleri I just merged #1674 . Could you please rebase and then merge when test are passing?

Fix #1610 unexpected field-error related to a well-formatted value example of a schema's boolean field using frictionless validate

@pierrecamilleri I just merged #1674 . Could you please rebase and then merge when test are passing?

Add HTMX to activity stream

Hello @aleeexgreeen ! As a first approach it is good but let's be a little bit more specific on the interaction to take full advantage of HTMX. The "problem" with...

Add HTMX to activity stream

@aleeexgreeen hello! You are using it correctly, there is a problem with the types definition that didn't got detected when merging the HTMX PR. @smotornyuk The problem is that our...

Add HTMX to activity stream

Perfect! Yes, there is no point in back porting this to CKAN 2.10 so let's go with your solution :) @aleeexgreeen could you add this changes to the PR and...

Add HTMX to activity stream

Thanks @aleeexgreeen for this PR! It looks great :)

Allow use of X-Accel-Redirect for downloaded resources

@EricSoroos what's the status of this? In terms of code what is missing is: - Proper entry in our `config_declarations.yaml` - A changelog file since it is a new addition.