Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

JeroenDeDauw

JeroenDeDauw closed an issue on ProfessionalWiki/WikibaseFacetedSearch
Create Elasticsearch fields
AC: * Add dedicated Elastic field per configured facet * example: `wbfs_P42` Related: * https://github.com/ProfessionalWiki/WikibaseFacetedSearch/issues/39
JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
Good candidate for own service Similar code in Semantic Wikibase: https://github.com/ProfessionalWiki/SemanticWikibase/blob/master/src/Translation/DataValueTranslator.php#L26

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
I suspect something like this will be nice: ```php private function getStatements(): StatementList { return $this->getStatementsLookup()->getStatements(); } private function getStatem...

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
Stating the obvious: this logic should be in instantiable class(es).

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
`Facet` does not seem needed in the name

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw deleted a branch ProfessionalWiki/WikibaseFacetedSearch

es-fields

JeroenDeDauw pushed 1 commit to master ProfessionalWiki/WikibaseFacetedSearch
  • Add initial search fields 05a739a

View on GitHub

JeroenDeDauw closed a pull request on ProfessionalWiki/WikibaseFacetedSearch
Add initial search fields
For https://github.com/ProfessionalWiki/WikibaseFacetedSearch/issues/49 Creates an Elasticsearch field for each configured facet. Example in Kibana: ![Screenshot_20241229_231759](https://git...
JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
Which makes me think we could go further than extracting a getFieldTypeForPropertyId method as suggested above, and instead make that a service.

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
I did some double checking about this approach, in particular if it's better to use Wikibase's internal type, and it seems good as it is. Here is what I did in Semantic Wikibase, which has a ful...

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
This line and the above try catch could also be in its own method: getFieldTypeForPropertyId

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
I suspect we can do `} catch ( Exception ) {` given we use modern PHP

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
`$name` can be computed closer to where it is used and the details of said computation could go into a dedicated private method.

View on GitHub

JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch

View on GitHub

JeroenDeDauw deleted a branch SemanticMediaWiki/SemanticMediaWiki

paladox-patch-1

JeroenDeDauw closed a pull request on SemanticMediaWiki/SemanticMediaWiki
Remove "SMWTestsDatabaseTestCase" unused import
<!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Tests** - Updated test configuration by removing a specific database test case import ...
JeroenDeDauw pushed 1 commit to master SemanticMediaWiki/SemanticMediaWiki
  • Remove "SMWTestsDatabaseTestCase" unused import (#5902) 7ed2e14

View on GitHub

JeroenDeDauw pushed 1 commit to master SemanticMediaWiki/SemanticWatchlist

View on GitHub

JeroenDeDauw deleted a branch SemanticMediaWiki/SemanticWatchlist

add-ci

JeroenDeDauw pushed 1 commit to master SemanticMediaWiki/SemanticWatchlist
  • Add CI (#113) * Add CI * Bumps MW requirements to 1.39+. * Requires PHP 7.4+ * Adds CI to run tests * Fixes te... 26a5b68

View on GitHub

JeroenDeDauw closed a pull request on SemanticMediaWiki/SemanticWatchlist
Add CI
* Bumps MW requirements to 1.39+. * Requires PHP 7.4+ * Adds CI to run tests * Fixes tests to support MW 1.39 * Adapts to changes done to the naming of files.
Load more