notifications-routing merge requestshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests2021-03-24T15:37:27+01:00https://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/51[FIX] DB mapping2021-03-24T15:37:27+01:00Carina Antunes[FIX] DB mappingCloses #
Changes:
- bump sqlalchemy to 1.3.
- remove reflection: Read more: https://docs.sqlalchemy.org/en/13/orm/extensions/automap.html#using-automap-with-explicit-declarationsCloses #
Changes:
- bump sqlalchemy to 1.3.
- remove reflection: Read more: https://docs.sqlalchemy.org/en/13/orm/extensions/automap.html#using-automap-with-explicit-declarationsCarina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/49BugFix: Load Schema from Configuration2021-03-24T09:34:52+01:00Igor JakovljevicBugFix: Load Schema from ConfigurationI tried to implement a [more elegant solution for loading schemas](https://stackoverflow.com/a/46482677), but it did not work. It did not load the schema correctly.
Changes:
- Load Schema from Configuraiton for DatabaseI tried to implement a [more elegant solution for loading schemas](https://stackoverflow.com/a/46482677), but it did not work. It did not load the schema correctly.
Changes:
- Load Schema from Configuraiton for DatabaseCarina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/48Fix email-daily job publisher extension2021-03-22T10:20:31+01:00Carina AntunesFix email-daily job publisher extensionCloses #
Changes:
-Closes #
Changes:
-https://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/47Fix poetry install2021-03-22T09:39:30+01:00Carina AntunesFix poetry install
- `poetry install --no-dev` is removing dependencies needed by poetry itself because it was installed with pip
> The problem here is, that poetry and the package are installed in the same environment. One should never mix the installat...
- `poetry install --no-dev` is removing dependencies needed by poetry itself because it was installed with pip
> The problem here is, that poetry and the package are installed in the same environment. One should never mix the installation method within one environment. Either use poetry or pip. But not both at the same time. Because they don't know each other and break things when a package gets installed or removed by one or another.
> When using virtualenvs.create false make sure poetry is installed with the installer script or as an alternative by pipx.
From: https://github.com/python-poetry/poetry/issues/2102#issuecomment-706740546
Solution: installing with the recommend ways or using `poetry install` in production.Carina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/43[#37] Bugfix: Channel's user list2021-03-19T16:53:44+01:00Dimitra Chatzichrysoudimitra.chatzichrysou@cern.ch[#37] Bugfix: Channel's user listChanges:
- Add `get_system_user` in datasource
- Update `get_channel_users` in `router.py`
Closes #37Changes:
- Add `get_system_user` in datasource
- Update `get_channel_users` in `router.py`
Closes #37PrototypeCarina AntunesIgor JakovljevicCaetan Tojeiro CarpenteCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/45Add OICD variable to local env2021-03-18T11:48:31+01:00Carina AntunesAdd OICD variable to local envCloses #
Changes:
- Add OICD variable to local envCloses #
Changes:
- Add OICD variable to local envhttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/44[#38] Push image to qa and prod image-streams2021-03-17T09:45:08+01:00Dimitra Chatzichrysoudimitra.chatzichrysou@cern.ch[#38] Push image to qa and prod image-streamsCloses #38Closes #38PrototypeMichal KolodziejskiCarina AntunesIgor JakovljevicCaetan Tojeiro CarpenteMichal Kolodziejskihttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/40[#31] Daily feed job2021-03-11T13:58:42+01:00Carina Antunes[#31] Daily feed jobCloses #31
The following discussion from !37 should be addressed:
- [x] @crdeoliv started a [discussion](https://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/37#note_4215266): (+2 comments)
> in pytho...Closes #31
The following discussion from !37 should be addressed:
- [x] @crdeoliv started a [discussion](https://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/37#note_4215266): (+2 comments)
> in python attributes follow the convention lower_case_with_underscores, also add type hints pleaseCarina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/42[HOTFIX] Fix daily notifications and uuids refactor2021-03-01T19:12:55+01:00Carina Antunes[HOTFIX] Fix daily notifications and uuids refactorChanges:
- Fix changes from UUID refactor and new Daily Notifications TableChanges:
- Fix changes from UUID refactor and new Daily Notifications TableCarina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/41[#36] Refactor ids: use uuid (clone)2021-02-26T19:00:35+01:00Emmanuel Ormancey[#36] Refactor ids: use uuid (clone)Closes #
Changes:
-
Closes #36Closes #
Changes:
-
Closes #36PrototypeEmmanuel OrmanceyEmmanuel Ormanceyhttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/37[#30] Implement daily preferences logic2021-02-25T16:18:56+01:00Igor Jakovljevic[#30] Implement daily preferences logicCloses #30Closes #30Michal KolodziejskiCarina AntunesCaetan Tojeiro CarpenteMichal Kolodziejskihttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/38"[PUSH] Route to Safari push specific consumer"2021-02-25T15:56:45+01:00Emmanuel Ormancey"[PUSH] Route to Safari push specific consumer"Closes #
Changes:
- Updated routing based on device type and subtype
- Added route to safaripush new consumer
- Added new consumer type in activemq definition
Closes #35Closes #
Changes:
- Updated routing based on device type and subtype
- Added route to safaripush new consumer
- Added new consumer type in activemq definition
Closes #35PrototypeEmmanuel OrmanceyCarina AntunesIgor JakovljevicEmmanuel Ormanceyhttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/36[#19] Standardize project: linting, poetry, kubectl2021-02-05T11:07:47+01:00Carina Antunes[#19] Standardize project: linting, poetry, kubectlCloses #19
Changes:
- Switch to poetry
- Make linter mandatory
- Fix lint errors
- Remove kubectl setupCloses #19
Changes:
- Switch to poetry
- Make linter mandatory
- Fix lint errors
- Remove kubectl setupMichal KolodziejskiIgor JakovljevicCaetan Tojeiro CarpenteMichal Kolodziejskihttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/35[#33][Target Devices] Resolve "Route LIVE notifications according to preferen...2021-02-01T20:02:32+01:00Emmanuel Ormancey[#33][Target Devices] Resolve "Route LIVE notifications according to preferences"Closes #
Changes:
- sends to the appropriate queues based on preference type and target devices
- LIVE + Mail and Webpush are implemented for now.
- implements the global preference toggle on channels ('disabledChannels' manytomany p...Closes #
Changes:
- sends to the appropriate queues based on preference type and target devices
- LIVE + Mail and Webpush are implemented for now.
- implements the global preference toggle on channels ('disabledChannels' manytomany preference column)
- added support for aesgcm optional encoding
- added support for link and imgurl in webpush
Closes #33PrototypeEmmanuel OrmanceyCarina AntunesEmmanuel Ormanceyhttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/34[#32] Implement send to browser queue2021-01-18T15:21:41+01:00Emmanuel Ormancey[#32] Implement send to browser queueCloses #
Changes:
- extracts the list of devices for the target user
- for LIVE + Device=BROWSER sends back np.webpush message with the target device (browser) token
Closes #32Closes #
Changes:
- extracts the list of devices for the target user
- for LIVE + Device=BROWSER sends back np.webpush message with the target device (browser) token
Closes #32PrototypeMichal KolodziejskiCarina AntunesIgor JakovljevicMichal Kolodziejskihttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/33[BUGFIX] Channel name property2020-11-18T13:39:10+01:00Carina Antunes[BUGFIX] Channel name propertyCaetan Tojeiro CarpenteCaetan Tojeiro Carpentehttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/32Improve logging2020-11-18T13:27:22+01:00Dimitra Chatzichrysoudimitra.chatzichrysou@cern.chImprove loggingChanges:
- Update `logging.yaml`
- Update `create_app` in `app.py`Changes:
- Update `logging.yaml`
- Update `create_app` in `app.py`Carina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/31[11 from consumer]Improve-Email-Template-Channel-Name2020-11-16T19:09:38+01:00Caetan Tojeiro Carpente[11 from consumer]Improve-Email-Template-Channel-NameAdded Channel name for consumer emailsAdded Channel name for consumer emailsCarina AntunesCarina Antuneshttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/30[#29] Send to email queue with header persistent2020-11-13T10:45:05+01:00Dimitra Chatzichrysoudimitra.chatzichrysou@cern.ch[#29] Send to email queue with header persistentChanges:
- Add `publisher.py` to extend `send` to pass headers parameter
- Update `app.py` to use extended `Publisher`
Closes #29Changes:
- Add `publisher.py` to extend `send` to pass headers parameter
- Update `app.py` to use extended `Publisher`
Closes #29PrototypeMichal KolodziejskiCarina AntunesMichal Kolodziejskihttps://gitlab.cern.ch/push-notifications/notifications-routing/-/merge_requests/28[#20] Configure Sentry2020-11-12T18:06:42+01:00Dimitra Chatzichrysoudimitra.chatzichrysou@cern.ch[#20] Configure SentryCloses #20Closes #20PrototypeMichal KolodziejskiCarina AntunesMichal Kolodziejski