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

Invite User fixes #19356

Closed
wants to merge 85 commits into from
Closed

Invite User fixes #19356

wants to merge 85 commits into from

Conversation

peterhashair
Copy link
Contributor

@peterhashair peterhashair commented Jun 14, 2022

Description:

Follow-up fixes for PR #18868

  • Fix the token, update the table structure, and put the invite token into the user table
  • Fix the VUE USER type
  • Fix some UI test
  • add terms and condition dynamic setting from the general setting
  • UI update for the expired time (when the token expired in 3 days, the time will display, otherwise will just display pending)
  • active log for who initial fire the invite (add post events)
  • Only superUser can alternate the invite

Review

Peter and others added 30 commits June 8, 2022 13:29
update token and etc
update vue
update vue error
update a error
update table and fields
update tests
* add scroll to live table

add scroll to live table

* add tests

add tests

* update tests

update tests

* update tests

update tests

* Update Dashboard_spec.js

show full image

* update tests

update tests

* add timeout and screenshots

add timeout and screenshots

* update scroll bar

update scroll bar

* update screenshot

update screenshot

* Rerun UI tests

* Increase UI screenshot delay to fix random failure

* update ui

update ui

Co-authored-by: sgiehl <stefan@matomo.org>
Co-authored-by: Ben <ben.burgess@innocraft.com>
update tests
update tests
Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
* Adds first bunch of referrer attribution tests

* simplify tests

* refactor tests to use data provider, so its easier adding new tests

* Add new test cases

* Refactor tests into automically created test cases

* implement tests for campaign handling

* Adjust tests so they pass
* Include conversion attribution in visitor details

* updates expected test files

* submodule updates
update screenshot
update insert
update tests and screeenshots
update php cs
update php cs
Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
update tests
github-actions bot and others added 28 commits June 17, 2022 10:32
Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
* Adds first bunch of referrer attribution tests

* simplify tests

* refactor tests to use data provider, so its easier adding new tests

* Add new test cases

* Refactor tests into automically created test cases

* implement tests for campaign handling

* Adjust tests so they pass
* Include conversion attribution in visitor details

* updates expected test files

* submodule updates
Co-authored-by: sgiehl <sgiehl@users.noreply.github.com>
update tests
update token expired and some checks
update type
update token and etc
update a error
update table and fields
update tests and screeenshots
update php cs
update token expired and some checks
update type
update inivitation
@peterhashair peterhashair deleted the inviteuserfix branch June 16, 2022 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants