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

Development to main #102

Merged
merged 39 commits into from
Oct 16, 2024
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
34f5e4d
Merge pull request #91 from ConductionNL/feature/AXCVDWOF-12/public-p…
WilcoLouwerse Oct 14, 2024
11f711c
Merge remote-tracking branch 'origin/feature/DIMOC-322/configuration-…
WilcoLouwerse Oct 14, 2024
5eba786
Removed source checks on publicationType and fixed eigenschappen edit
remko48 Oct 14, 2024
7c2d12b
Merge remote-tracking branch 'origin/development' into feature/DIMOC-…
remko48 Oct 14, 2024
1605c93
Updated internal search to use publications endpoint
remko48 Oct 14, 2024
4d17536
Let's start with some basic fixes...
WilcoLouwerse Oct 14, 2024
c213ad9
Fix old directoryController update function
WilcoLouwerse Oct 14, 2024
6acdbb0
Request registers (with extend) from openregister
rjzondervan Oct 14, 2024
5e6ffb3
Added missing synchronise function to DirectoryService
WilcoLouwerse Oct 14, 2024
67cd0ba
master to dev
remko48 Oct 14, 2024
0a74fda
Merge pull request #97 from ConductionNL/merge-main-to-dev
remko48 Oct 14, 2024
3840e44
Merge remote-tracking branch 'origin/development' into feature/DIMOC-…
remko48 Oct 14, 2024
5eb13c2
Merge remote-tracking branch 'origin/development' into feature/DIMOC-…
remko48 Oct 14, 2024
7482327
Merge branch 'feature/DIMOC-330/Get-schemas-from-openregister' into f…
remko48 Oct 14, 2024
1299881
We don't want to set a default Source for PublicationType
WilcoLouwerse Oct 15, 2024
b0aea90
Merge pull request #95 from ConductionNL/feature/DIMOC-330/Get-schema…
rjzondervan Oct 15, 2024
26f3284
Merge branch 'development' into feature/DIMOC-322/update-dev-before-m…
WilcoLouwerse Oct 15, 2024
a4aa91a
fixed all bugs
SudoThijn Oct 15, 2024
19bdcde
Updated config page
remko48 Oct 15, 2024
cbc252f
cleanup
remko48 Oct 15, 2024
fba910b
More cleanup
remko48 Oct 15, 2024
5487920
Requested changes
remko48 Oct 15, 2024
dec1629
typo fixes
SudoThijn Oct 15, 2024
240d233
Merge pull request #99 from ConductionNL/feature/DIMOC-322/config-page
remko48 Oct 15, 2024
e847d8d
More cleanup
WilcoLouwerse Oct 15, 2024
31bcf0a
Fixes for updating known Listings after Catalog is added/updated
WilcoLouwerse Oct 15, 2024
66cac3d
Merge pull request #94 from ConductionNL/feature/DIMOC-329/search-page
remko48 Oct 15, 2024
3514d72
Merge pull request #100 from ConductionNL/feature/DIMOC-335/attachmen…
remko48 Oct 15, 2024
352f7f6
Merge remote-tracking branch 'origin/development' into feature/DIMOC-…
remko48 Oct 15, 2024
64fdd93
Merge pull request #101 from ConductionNL/feature/DIMOC-327/publicati…
remko48 Oct 15, 2024
236c4a8
Fix delete in objectService
WilcoLouwerse Oct 15, 2024
f10d2bc
Fix delete in objectService 2.0
WilcoLouwerse Oct 15, 2024
54962c1
Final Fixes for updating known Listings after Catalog is added/updated
WilcoLouwerse Oct 15, 2024
b26e5ca
Merge remote-tracking branch 'origin/development' into feature/DIMOC-…
remko48 Oct 16, 2024
300ee2c
Small frontend changes: Added Empty list headers and fixed deletion g…
remko48 Oct 16, 2024
6b5a503
More small fixes and publish publication bugfix
remko48 Oct 16, 2024
8e2a4c1
Added requested changes
remko48 Oct 16, 2024
8f0b7d2
cleanup
remko48 Oct 16, 2024
86f6ce4
Merge pull request #98 from ConductionNL/feature/DIMOC-322/update-dev…
remko48 Oct 16, 2024
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Removed source checks on publicationType and fixed eigenschappen edit
  • Loading branch information
remko48 committed Oct 14, 2024
commit 5eba786cbe29d87ac7d59d114dd8c2faa5c626b1
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ export default {
methods: {
deleteCatalogiPublicationType() {
const publicationTypeArray = catalogiStore.catalogiItem?.publicationTypes
.filter((source) => source !== (publicationTypeStore.publicationTypeItem?.source || publicationTypeStore.publicationTypeItem?.id))
.filter((id) => id !== publicationTypeStore.publicationTypeItem?.id)

const catalogiItem = new Catalogi({
...catalogiStore.catalogiItem,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ export default {
publicationTypeStore.getAllPublicationTypes()
.then(({ response, data }) => {
const filteredData = data.filter((publicationType) =>
!publicationTypeList.includes(publicationType?.source || publicationType?.id),
!publicationTypeList.includes(publicationType?.id),
)

this.publicationTypes = {
Expand All @@ -146,7 +146,7 @@ export default {
this.loading = true
this.error = false

this.catalogiItem.publicationTypes.push(this.publicationTypes.value.source !== '' ? this.publicationTypes.value.source : this.publicationTypes.value.id)
this.catalogiItem.publicationTypes.push(this.publicationTypes.value.id)

const newCatalogiItem = new Catalogi({
...this.catalogiItem,
Expand Down
10 changes: 3 additions & 7 deletions src/modals/publication/AddPublicationModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@ export default {
// step 2: get the full publication types from the publicationTypeIds
const filteredPublicationType = this.publicationTypeList
.filter((publicationType) => {
const publicationTypeId = publicationType.source || publicationType.id
const publicationTypeId = publicationType.id
return selectedCatalogus.publicationTypes.includes(publicationTypeId.toString())
})

Expand All @@ -268,9 +268,7 @@ export default {
const testClass = new Publication({
...this.publication,
catalog: this.catalogi.value?.id,
publicationType: this.publicationType?.value?.source !== ''
? this.publicationType?.value?.source
: this.publicationType?.value?.id,
publicationType: this.publicationType?.value?.id,
published: this.publication.published !== '' ? new Date(this.publication.published).toISOString() : new Date().toISOString(),
})

Expand Down Expand Up @@ -363,9 +361,7 @@ export default {
const publicationItem = new Publication({
...this.publication,
catalog: this.catalogi?.value?.id,
publicationType: this.publicationType?.value?.source !== ''
? this.publicationType?.value?.source
: this.publicationType?.value?.id,
publicationType: this.publicationType?.value?.id,
published: this.publication.published !== '' ? new Date(this.publication.published).toISOString() : new Date().toISOString(),
})

Expand Down
3 changes: 2 additions & 1 deletion src/modals/publicationData/EditPublicationDataModal.vue
Original file line number Diff line number Diff line change
Expand Up @@ -264,7 +264,8 @@ export default {
console.log('activePublicationType', this.publicationTypeList)

// get the publication type linked to this publication
const activePublicationType = this.publicationTypeList.find((publicationType) => publicationType.source === this.publication.publicationType)
const activePublicationType = this.publicationTypeList.find((publicationType) => publicationType.id.toString() === this.publication.publicationType.toString(),
)
// get all the properties as an array of values (key is not needed as the comparison is done by title)
const publicationTypeProperties = Object.values(activePublicationType.properties)
// get the publication type properties with the same title as the publicationDataKey
Expand Down
16 changes: 8 additions & 8 deletions src/views/catalogi/CatalogiDetails.vue
Original file line number Diff line number Diff line change
Expand Up @@ -90,26 +90,26 @@ import { catalogiStore, publicationTypeStore, navigationStore, organizationStore
<BTabs content-class="mt-3" justified>
<BTab title="Publicatietypes">
<div v-if="catalogiStore.catalogiItem?.publicationTypes.length > 0 && !publicationTypeLoading">
<NcListItem v-for="(url, i) in catalogiStore.catalogiItem?.publicationTypes"
:key="url + i"
:name="filteredPublicationType(url)?.title || 'loading...'"
<NcListItem v-for="(id, i) in catalogiStore.catalogiItem?.publicationTypes"
:key="id + i"
:name="filteredPublicationType(id)?.title || 'loading...'"
:bold="false"
:force-display-actions="true">
<template #icon>
<FileTreeOutline disable-menu
:size="44" />
</template>
<template #subname>
{{ filteredPublicationType(url)?.description }}
{{ filteredPublicationType(id)?.description }}
</template>
<template #actions>
<NcActionButton @click="publicationTypeStore.setPublicationTypeItem(filteredPublicationType(url)); navigationStore.setSelected('publicationType')">
<NcActionButton @click="publicationTypeStore.setPublicationTypeItem(filteredPublicationType(id)); navigationStore.setSelected('publicationType')">
<template #icon>
<OpenInApp :size="20" />
</template>
Bekijk publicatietype
</NcActionButton>
<NcActionButton @click="publicationTypeStore.setPublicationTypeItem(filteredPublicationType(url)); navigationStore.setDialog('deleteCatalogiPublicationType')">
<NcActionButton @click="publicationTypeStore.setPublicationTypeItem(filteredPublicationType(id)); navigationStore.setDialog('deleteCatalogiPublicationType')">
<template #icon>
<Delete :size="20" />
</template>
Expand Down Expand Up @@ -237,10 +237,10 @@ export default {
if (loading) { this.organizationLoading = false }
})
},
filteredPublicationType(source) {
filteredPublicationType(id) {
if (this.publicationTypeLoading) return null
return publicationTypeStore.publicationTypeList.filter((publicationType) => {
return publicationType?.source ? publicationType?.source === source : publicationType?.id === source
return publicationType?.id === id
})[0]
},
goToOrganization() {
Expand Down
2 changes: 1 addition & 1 deletion src/views/publications/PublicationDetail.vue
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,7 @@ import { ref } from 'vue'
{{ value }}
</template>
<template #actions>
<NcActionButton :disabled="true" @click="editPublicationDataItem(key)">
<NcActionButton @click="editPublicationDataItem(key)">
<template #icon>
<Pencil :size="20" />
</template>
Expand Down