Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
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/39JeroenDeDauw 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
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
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...
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
Stating the obvious: this logic should be in instantiable class(es).
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
`Facet` does not seem needed in the name
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
JeroenDeDauw pushed 1 commit to master ProfessionalWiki/WikibaseFacetedSearch
- Add initial search fields 05a739a
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.
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
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...
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
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
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
JeroenDeDauw created a review comment on a pull request on ProfessionalWiki/WikibaseFacetedSearch
I suspect we can do `} catch ( Exception ) {` given we use modern PHP
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
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.
JeroenDeDauw created a review on a pull request on ProfessionalWiki/WikibaseFacetedSearch
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
JeroenDeDauw pushed 1 commit to master SemanticMediaWiki/SemanticWatchlist
- Fix README badges f18823a
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