Adjusted notification token and device id size limits
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed All Stories
All Stories
All Stories
Aug 18 2021
Aug 18 2021
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
Included console tests
Removed assemble changes
machniak requested changes to D2674: NGINX Controller, 2fa for client connections and companion app support.
vanmeeuwen committed rKb70dbda55bad: Allow search with transaction and customer IDs (authored by vanmeeuwen).
Allow search with transaction and customer IDs
- Add a search for a wallet id
- Add tests for the api calls
- Here be a method not a property
- Relations are properties
mollekopf added a reviewer for D2728: Set errors if something fails during append: Restricted Project.
I added this while using the migration code because sometimes we would just silently fail.
This is the code as we used it for the migration, I'm not looking to invest a lot of time in this until we decide to actually use it elsewhere.
mollekopf added inline comments to D2674: NGINX Controller, 2fa for client connections and companion app support.
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
Addressed comments, moved to services. url
Moved to uuids for clients
All comments resolved.
- Commentary, safety on settings not found
back to oauth proxy client
Change passport over to using UUIDs
Laravel Passport support
Because both these additional searches are mollie-specific maybe this code should be abstracted-on/moved-to the Providers/PaymentProvider. E.g. isPaymentId(), isCustomerId(), findWalletByPaymentOrCustomerId() or sth like that.
Aug 17 2021
Aug 17 2021
machniak requested changes to D2674: NGINX Controller, 2fa for client connections and companion app support.
mollekopf added inline comments to D2674: NGINX Controller, 2fa for client connections and companion app support.
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
Addressed comments.
Merge branch 'master' into dev/kolabmeet
Basic PowerDNS functionality
machniak committed rKd30fd0cfff6a: Refresh package-lock.json, fix problem with frappe-charts styles integration (authored by machniak).
Refresh package-lock.json, fix problem with frappe-charts styles integration
mollekopf added inline comments to D2674: NGINX Controller, 2fa for client connections and companion app support.
mollekopf committed rK52b13c22685f: NGINX Controller, 2fa for client connections and companion app support (authored by mollekopf).
NGINX Controller, 2fa for client connections and companion app support
Laravel Passport support
Fix linting issues, mostly indentation
Bootstrap v5
- Describe $domain
- Rebase on master
vanmeeuwen requested changes to D2674: NGINX Controller, 2fa for client connections and companion app support.
One last issue. Other than that it looks good. I didn't test the docker part, I guess we'll need some intergration tests later.
vanmeeuwen committed rK9c0964980d2f: Add possibility to change system currency (authored by machniak).
Add possibility to change system currency
vanmeeuwen committed rKc282a9c473bd: Consistently use website_domain for api (authored by mollekopf).
Consistently use website_domain for api
Greylist alias test
NGINX docker container
Add PGP keys generator
Fix kolab container setup
- Minimal validation.
- Just use $domain->name everywhere
- Disable domain observer events in turn bumping the serial
- Make use of the records relation
- Fix typo, test domain creation/observer
Incoming! ;-)
machniak added inline comments to D2674: NGINX Controller, 2fa for client connections and companion app support.
machniak added inline comments to D2674: NGINX Controller, 2fa for client connections and companion app support.
machniak requested changes to D2674: NGINX Controller, 2fa for client connections and companion app support.
I miss tests for console commands. I get they are simple and CLI has the worse coverage, but still it shouldn't be that hard.
Aug 16 2021
Aug 16 2021
Userdata migration for pst import
mollekopf committed rKc8a9d1936cc4: Set errors if something fails during append (authored by mollekopf).
Set errors if something fails during append
Fix kolab container setup
Ready for merge
Merged into D2413
Minor comments just after the code review, no testing and closer look on it yet.
Merged into D2494
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
Addressed comments
Include writing out the passport keys in the assemble script
- Remove another irrelevant change
- Remove useless change
mollekopf updated the diff for D2674: NGINX Controller, 2fa for client connections and companion app support.
Update diff to latest state
mollekopf added a reviewer for D2710: Seed oauth public/private keys from env variable in assemble script: Restricted Project.