diff --git a/backend/src/apis/KlicktippController.ts b/backend/src/apis/KlicktippController.ts index ebc3d6cef..a4b5b6be7 100644 --- a/backend/src/apis/KlicktippController.ts +++ b/backend/src/apis/KlicktippController.ts @@ -91,3 +91,18 @@ export const getKlicktippTagMap = async () => { } return '' } + +export const addFieldsToSubscriber = async ( + email: string, + fields: any = {}, + newemail = '', + newsmsnumber = '', +) => { + if (!CONFIG.KLICKTIPP) return true + const isLogin = await loginKlicktippUser() + if (isLogin) { + const subscriberId = await klicktippConnector.subscriberSearch(email) + return klicktippConnector.subscriberUpdate(subscriberId, fields, newemail, newsmsnumber) + } + return false +} diff --git a/federation/Dockerfile b/federation/Dockerfile index 959252d29..81b95010e 100644 --- a/federation/Dockerfile +++ b/federation/Dockerfile @@ -86,7 +86,7 @@ RUN cd ../database && yarn run build FROM build as test # Run command -CMD /bin/sh -c "yarn run start" +CMD /bin/sh -c "yarn run dev" ################################################################################## # PRODUCTION (Does contain only "binary"- and static-files to reduce image size) # diff --git a/frontend/package.json b/frontend/package.json index 8515ca209..f07284da6 100755 --- a/frontend/package.json +++ b/frontend/package.json @@ -44,7 +44,7 @@ "graphql": "^15.5.1", "identity-obj-proxy": "^3.0.0", "jest": "^26.6.3", - "jest-canvas-mock": "^2.3.1", + "jest-canvas-mock": "^2.5.0", "jwt-decode": "^3.1.2", "portal-vue": "^2.1.7", "prettier": "^2.2.1", diff --git a/frontend/src/components/LanguageSwitch.spec.js b/frontend/src/components/LanguageSwitch.spec.js index 7f37c535a..6c733de7d 100644 --- a/frontend/src/components/LanguageSwitch.spec.js +++ b/frontend/src/components/LanguageSwitch.spec.js @@ -15,7 +15,7 @@ describe('LanguageSwitch', () => { let wrapper const state = { - email: 'he@ho.he', + gradidoID: 'current-user-id', language: null, } diff --git a/frontend/src/components/LanguageSwitch.vue b/frontend/src/components/LanguageSwitch.vue index 9d901ae9f..38194ba9c 100644 --- a/frontend/src/components/LanguageSwitch.vue +++ b/frontend/src/components/LanguageSwitch.vue @@ -31,7 +31,7 @@ export default { async saveLocale(locale) { // if (this.$i18n.locale === locale) return this.setLocale(locale) - if (this.$store.state.email) { + if (this.$store.state.gradidoID) { this.$apollo .mutate({ mutation: updateUserInfos, diff --git a/frontend/src/components/LanguageSwitch2.spec.js b/frontend/src/components/LanguageSwitch2.spec.js index 0d2b485ec..e7ff6e8c6 100644 --- a/frontend/src/components/LanguageSwitch2.spec.js +++ b/frontend/src/components/LanguageSwitch2.spec.js @@ -15,7 +15,7 @@ describe('LanguageSwitch', () => { let wrapper const state = { - email: 'he@ho.he', + gradidoID: 'current-user-id', language: null, } diff --git a/frontend/src/components/LanguageSwitch2.vue b/frontend/src/components/LanguageSwitch2.vue index d398d2fe0..46b2a5277 100644 --- a/frontend/src/components/LanguageSwitch2.vue +++ b/frontend/src/components/LanguageSwitch2.vue @@ -59,7 +59,7 @@ export default { async saveLocale(locale) { if (this.$i18n.locale === locale) return this.setLocale(locale) - if (this.$store.state.email) { + if (this.$store.state.gradidoID) { this.$apollo .mutate({ mutation: updateUserInfos, diff --git a/frontend/src/components/Menu/Navbar.spec.js b/frontend/src/components/Menu/Navbar.spec.js index 1e05df71d..a942c2644 100644 --- a/frontend/src/components/Menu/Navbar.spec.js +++ b/frontend/src/components/Menu/Navbar.spec.js @@ -20,7 +20,7 @@ const mocks = { state: { firstName: 'Testy', lastName: 'User', - email: 'testy.user@example.com', + gradidoID: 'current-user-id', }, }, } @@ -64,8 +64,8 @@ describe('AuthNavbar', () => { ) }) - it('has the email address', () => { - // expect(wrapper.find('div.small:nth-child(2)').text()).toBe(wrapper.vm.$store.state.email) + // I think this should be username + it.skip('has the email address', () => { expect(wrapper.find('div[data-test="navbar-item-email"]').text()).toBe( wrapper.vm.$store.state.email, ) diff --git a/frontend/src/components/TransactionRows/AmountAndNameRow.spec.js b/frontend/src/components/TransactionRows/AmountAndNameRow.spec.js index 172f5f401..747cefe64 100644 --- a/frontend/src/components/TransactionRows/AmountAndNameRow.spec.js +++ b/frontend/src/components/TransactionRows/AmountAndNameRow.spec.js @@ -39,37 +39,5 @@ describe('AmountAndNameRow', () => { expect(wrapper.find('div.gdd-transaction-list-item-name').find('a').exists()).toBe(false) }) }) - - describe('with linked user', () => { - beforeEach(async () => { - await wrapper.setProps({ - linkedUser: { firstName: 'Bibi', lastName: 'Bloxberg', email: 'bibi@bloxberg.de' }, - }) - }) - - it('has a link with first and last name', () => { - expect(wrapper.find('div.gdd-transaction-list-item-name').text()).toBe('Bibi Bloxberg') - }) - - it('has a link', () => { - expect(wrapper.find('div.gdd-transaction-list-item-name').find('a').exists()).toBe(true) - }) - - describe('click link', () => { - beforeEach(async () => { - await wrapper.find('div.gdd-transaction-list-item-name').find('a').trigger('click') - }) - - it('emits set tunneled email', () => { - expect(wrapper.emitted('set-tunneled-email')).toEqual([['bibi@bloxberg.de']]) - }) - - it('pushes the route with query for email', () => { - expect(mocks.$router.push).toBeCalledWith({ - path: '/send', - }) - }) - }) - }) }) }) diff --git a/frontend/src/components/TransactionRows/AmountAndNameRow.vue b/frontend/src/components/TransactionRows/AmountAndNameRow.vue index eb68d9f37..530fdf0e8 100644 --- a/frontend/src/components/TransactionRows/AmountAndNameRow.vue +++ b/frontend/src/components/TransactionRows/AmountAndNameRow.vue @@ -10,21 +10,7 @@
- - - {{ itemText }} - - - {{ itemText }} - - {{ $t('via_link') }} - - + {{ text }}
@@ -38,31 +24,9 @@ export default { type: String, required: true, }, - linkedUser: { - type: Object, - required: false, - }, text: { type: String, - required: false, - }, - linkId: { - type: Number, - required: false, - default: null, - }, - }, - methods: { - tunnelEmail() { - this.$emit('set-tunneled-email', this.linkedUser.email) - this.$router.push({ path: '/send' }) - }, - }, - computed: { - itemText() { - return this.linkedUser - ? this.linkedUser.firstName + ' ' + this.linkedUser.lastName - : this.text + required: true, }, }, } diff --git a/frontend/src/components/UserSettings/UserNewsletter.spec.js b/frontend/src/components/UserSettings/UserNewsletter.spec.js index b211ec169..7ae3ddd7c 100644 --- a/frontend/src/components/UserSettings/UserNewsletter.spec.js +++ b/frontend/src/components/UserSettings/UserNewsletter.spec.js @@ -18,7 +18,6 @@ describe('UserCard_Newsletter', () => { $store: { state: { language: 'de', - email: 'peter@lustig.de', newsletterState: true, }, commit: storeCommitMock, diff --git a/frontend/src/graphql/mutations.js b/frontend/src/graphql/mutations.js index 65bdd7497..802ea1818 100644 --- a/frontend/src/graphql/mutations.js +++ b/frontend/src/graphql/mutations.js @@ -145,7 +145,6 @@ export const login = gql` mutation($email: String!, $password: String!, $publisherId: Int) { login(email: $email, password: $password, publisherId: $publisherId) { gradidoID - email firstName lastName language diff --git a/frontend/src/graphql/queries.js b/frontend/src/graphql/queries.js index d0cbc145d..a21117ac2 100644 --- a/frontend/src/graphql/queries.js +++ b/frontend/src/graphql/queries.js @@ -3,7 +3,7 @@ import gql from 'graphql-tag' export const verifyLogin = gql` query { verifyLogin { - email + gradidoID firstName lastName language @@ -40,7 +40,6 @@ export const transactionsQuery = gql` firstName lastName gradidoID - email } decay { decay @@ -102,9 +101,9 @@ export const queryTransactionLink = gql` redeemedAt deletedAt user { + gradidoID firstName publisherId - email } } ... on ContributionLink { diff --git a/frontend/src/layouts/DashboardLayout.spec.js b/frontend/src/layouts/DashboardLayout.spec.js index 9f68199da..a2a666591 100644 --- a/frontend/src/layouts/DashboardLayout.spec.js +++ b/frontend/src/layouts/DashboardLayout.spec.js @@ -43,7 +43,6 @@ const mocks = { $store: { dispatch: storeDispatchMock, state: { - email: 'user@example.org', publisherId: 123, firstName: 'User', lastName: 'Example', @@ -260,34 +259,6 @@ describe('DashboardLayout', () => { }) }) - describe.skip('elopage URI', () => { - describe('user has no publisher ID and no elopage', () => { - beforeEach(() => { - mocks.$store.state.publisherId = null - mocks.$store.state.hasElopage = false - wrapper = Wrapper() - }) - - it('links to basic-de', () => { - expect(wrapper.vm.elopageUri).toBe( - 'https://elopage.com/s/gradido/basic-de/payment?locale=en&prid=111&pid=2896&firstName=User&lastName=Example&email=user@example.org', - ) - }) - }) - - describe('user has elopage', () => { - beforeEach(() => { - mocks.$store.state.publisherId = '123' - mocks.$store.state.hasElopage = true - wrapper = Wrapper() - }) - - it('links to sign in for elopage', () => { - expect(wrapper.vm.elopageUri).toBe('https://elopage.com/s/gradido/sign_in?locale=en') - }) - }) - }) - describe.skip('admin method', () => { const windowLocationMock = jest.fn() beforeEach(() => { diff --git a/frontend/src/pages/TransactionLink.spec.js b/frontend/src/pages/TransactionLink.spec.js index adbb25226..0b9cbb5ef 100644 --- a/frontend/src/pages/TransactionLink.spec.js +++ b/frontend/src/pages/TransactionLink.spec.js @@ -32,7 +32,7 @@ apolloQueryMock.mockResolvedValue({ validUntil: transactionLinkValidExpireDate(), redeemedAt: '2022-03-18T10:08:43.000Z', deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -44,7 +44,7 @@ const mocks = { state: { token: null, tokenTime: null, - email: 'bibi@bloxberg.de', + gradidoID: 'current-user-id', }, }, $apollo: { @@ -101,7 +101,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: '2022-03-18T10:08:43.000Z', deletedAt: now, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -132,7 +132,7 @@ describe('TransactionLink', () => { validUntil: '2020-03-18T10:08:43.000Z', redeemedAt: '2022-03-18T10:08:43.000Z', deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -163,7 +163,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: '2022-03-18T10:08:43.000Z', deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -195,7 +195,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: null, deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -239,7 +239,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: null, deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'current-user-id' }, }, }, }) @@ -275,7 +275,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: null, deletedAt: null, - user: { firstName: 'Peter', publisherId: 0, email: 'peter@listig.de' }, + user: { firstName: 'Peter', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) @@ -351,7 +351,7 @@ describe('TransactionLink', () => { validUntil: transactionLinkValidExpireDate(), redeemedAt: null, deletedAt: null, - user: { firstName: 'Bibi', publisherId: 0, email: 'bibi@bloxberg.de' }, + user: { firstName: 'Bibi', publisherId: 0, gradidoID: 'other-user-id' }, }, }, }) diff --git a/frontend/src/pages/TransactionLink.vue b/frontend/src/pages/TransactionLink.vue index 0aa4cfa7d..adc3cb50b 100644 --- a/frontend/src/pages/TransactionLink.vue +++ b/frontend/src/pages/TransactionLink.vue @@ -139,7 +139,7 @@ export default { if (this.tokenExpiresInSeconds < 5) return `LOGGED_OUT` // logged in, nicht berechtigt einzulösen, eigener link - if (this.linkData.user && this.$store.state.email === this.linkData.user.email) { + if (this.linkData.user && this.$store.state.gradidoID === this.linkData.user.gradidoID) { return `SELF_CREATOR` } diff --git a/frontend/src/store/store.js b/frontend/src/store/store.js index 1cd874c06..436a2d32b 100644 --- a/frontend/src/store/store.js +++ b/frontend/src/store/store.js @@ -13,8 +13,8 @@ export const mutations = { localeChanged(language) state.language = language }, - email: (state, email) => { - state.email = email + gradidoID: (state, gradidoID) => { + state.gradidoID = gradidoID }, // username: (state, username) => { // state.username = username @@ -57,7 +57,7 @@ export const mutations = { export const actions = { login: ({ dispatch, commit }, data) => { - commit('email', data.email) + commit('gradidoID', data.gradidoID) commit('language', data.language) // commit('username', data.username) commit('firstName', data.firstName) @@ -71,8 +71,8 @@ export const actions = { }, logout: ({ commit, state }) => { commit('token', null) - commit('email', null) // commit('username', '') + commit('gradidoID', null) commit('firstName', '') commit('lastName', '') commit('newsletterState', null) @@ -95,8 +95,8 @@ try { }), ], state: { - email: '', language: null, + gradidoID: null, firstName: '', lastName: '', // username: '', diff --git a/frontend/src/store/store.test.js b/frontend/src/store/store.test.js index 33fedd562..cc4d6a284 100644 --- a/frontend/src/store/store.test.js +++ b/frontend/src/store/store.test.js @@ -22,7 +22,7 @@ i18n.locale = 'blubb' const { language, - email, + gradidoID, token, firstName, lastName, @@ -53,11 +53,11 @@ describe('Vuex store', () => { }) }) - describe('email', () => { - it('sets the state of email', () => { - const state = { email: 'nobody@knows.tv' } - email(state, 'someone@there.is') - expect(state.email).toEqual('someone@there.is') + describe('gradidoID', () => { + it('sets the state of gradidoID', () => { + const state = { gradidoID: 'old-id' } + gradidoID(state, 'new-id') + expect(state.gradidoID).toEqual('new-id') }) }) @@ -164,7 +164,7 @@ describe('Vuex store', () => { const commit = jest.fn() const state = {} const commitedData = { - email: 'user@example.org', + gradidoID: 'my-gradido-id', language: 'de', firstName: 'Peter', lastName: 'Lustig', @@ -183,9 +183,9 @@ describe('Vuex store', () => { expect(commit).toHaveBeenCalledTimes(10) }) - it('commits email', () => { + it('commits gradidoID', () => { login({ commit, state }, commitedData) - expect(commit).toHaveBeenNthCalledWith(1, 'email', 'user@example.org') + expect(commit).toHaveBeenNthCalledWith(1, 'gradidoID', 'my-gradido-id') }) it('commits language', () => { @@ -248,9 +248,9 @@ describe('Vuex store', () => { expect(commit).toHaveBeenNthCalledWith(1, 'token', null) }) - it('commits email', () => { + it('commits gradidoID', () => { logout({ commit, state }) - expect(commit).toHaveBeenNthCalledWith(2, 'email', null) + expect(commit).toHaveBeenNthCalledWith(2, 'gradidoID', null) }) it('commits firstName', () => { diff --git a/frontend/yarn.lock b/frontend/yarn.lock index 9e8fdd81d..7cc8e5fe5 100644 --- a/frontend/yarn.lock +++ b/frontend/yarn.lock @@ -8657,10 +8657,10 @@ javascript-stringify@^1.6.0: resolved "https://registry.yarnpkg.com/javascript-stringify/-/javascript-stringify-1.6.0.tgz#142d111f3a6e3dae8f4a9afd77d45855b5a9cce3" integrity sha1-FC0RHzpuPa6PSpr9d9RYVbWpzOM= -jest-canvas-mock@^2.3.1: - version "2.3.1" - resolved "https://registry.yarnpkg.com/jest-canvas-mock/-/jest-canvas-mock-2.3.1.tgz#9535d14bc18ccf1493be36ac37dd349928387826" - integrity sha512-5FnSZPrX3Q2ZfsbYNE3wqKR3+XorN8qFzDzB5o0golWgt6EOX1+emBnpOc9IAQ+NXFj8Nzm3h7ZdE/9H0ylBcg== +jest-canvas-mock@^2.5.0: + version "2.5.0" + resolved "https://registry.yarnpkg.com/jest-canvas-mock/-/jest-canvas-mock-2.5.0.tgz#3e60f87f77ddfa273cf8e7e4ea5f86fa827c7117" + integrity sha512-s2bmY2f22WPMzhB2YA93kiyf7CAfWAnV/sFfY9s48IVOrGmwui1eSFluDPesq1M+7tSC1hJAit6mzO0ZNXvVBA== dependencies: cssfontparser "^1.2.1" moo-color "^1.0.2"