Skip to content

Hack/tenant mock #3702

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 97 commits into from
Apr 26, 2019
Merged

Hack/tenant mock #3702

merged 97 commits into from
Apr 26, 2019

Conversation

matleppa
Copy link
Member

Closes #

Changes

Describe your changes here as a bulleted list:

  • change one
  • change two

Developer checklist

This checklist is to be completed by the PR developer:

  • Alternative solutions were compared/discussed before writing code
    • trade-offs with this solution are considered acceptable
  • Code in this PR adheres to the project styleguide
  • This pull request does not decrease project test coverage
  • If the code changes existing database collection(s), migration has been written
  • If UI texts are added or changed, all texts are internationalized

Reviewer checklist

Reviewed by: @username1

This list is to be completed by the pull request reviewer:

  • Code works as described/expected
  • Code seems to be error free
    • no browser console errors visible
    • no server console errors visible
    • passes CI build
  • Code is written in a way that promotes maintainability
    • easy to understand
    • well organized
    • follows project coding standards and conventions
    • well documented

@ghost ghost assigned matleppa Jan 25, 2019
@ghost ghost added the in progress label Jan 25, 2019
matleppa added 26 commits January 29, 2019 13:54
@matleppa
Show only datalist as autocomplete
a02d754
@matleppa
Fix in tenant toolbar
c4e7c7d
@matleppa
Changed ollk and feel of mock tenant
c9f9021 
- added addition of tenant users
- added listing of tenant users
@matleppa
User addition uses Session
62bf732 
- added checking taht a user must be selected
@matleppa
lint fixes
0a98b3a
@matleppa
Tenants are listed on page
22e6344
@matleppa
Removed unnecessary logging
c88da6b
@matleppa
New tenant can be added into list and a tenant can be remove from list
bd76c6a
@matleppa
Indicate missing role with '-'
6fe2be4
@matleppa
Indicate missing role with '-'
d2752c4
@matleppa
lint fixes
16e4ca2
@matleppa
Restructured
d2969ff
@matleppa
Added tenant configuration fields
ffb9192
@matleppa
Tenant url settings
f24fa41
@matleppa
Added description into tenant data + id in user data
15d4f9a
@matleppa
Changed tenant list and user list internally array of objects
0adb788
@matleppa
mocking answer from tenant manager
2a821fe
@matleppa
Correction in userlist fetch
bb29cfb
@matleppa
HTTP request synchronity check
6cffc07
@matleppa
Renamed field for tenant base path
6c7aa5e
@matleppa
Session data not stored as JSON
f2a7cd9
@matleppa
Removed token from Settings
7073d97
@matleppa
Conveyed number of tenants
f5ea135
@matleppa
Moved initialization of tenant list to method
d187335
@matleppa
Splitted autoruns for tenants and userlist
13df98f
@matleppa
Some localization
0276d7d
matleppa added 26 commits March 25, 2019 17:36
@matleppa
Cleaning up code
734ba59
@matleppa
lint fix
b1f0749
@matleppa
lint fixes
13c27bf
@matleppa
lint fixes
e525708
@matleppa
Cleaning up codebase
70d184b
@matleppa
Cleaned up code
e5f9205
@matleppa
Cleaning up code
1b41eec
@matleppa
New version of chart.js types
76c1130
@matleppa
Merge branch 'develop' into hack/tenant-mock
f7284a7
@matleppa
updated chart.js version
bd011ce
@matleppa
Merge branch 'hack/tenant-mock' of https://github.com/apinf/platform …
b1bd6c3 
…into hack/tenant-mock
@matleppa
Upgraded meteor version
9e0d979
@matleppa
Fix in chart.js import
efd88ef
@matleppa
Added spinner to indicate ongoing operation
9611461
@matleppa
lint fixes
6d71865
@matleppa
Notification about tenant user role changes
7ed48dd
@matleppa
Notify user always when removed, otherwise when indicated
3596043
@matleppa
Added email notification
2e4ea68
@matleppa
Localized notification message
2e8719c
@matleppa
lint fixes
536b64c
@matleppa
Show notification tick box only when mail is enabled
e9440c8
@matleppa
Revised comments
d7791f5
@matleppa
Comment coorection
faa369f
@matleppa
Lint fix
9982e76
@matleppa
Corrections after review
6db9b00
@matleppa
Corrections after review
c7a1b6e
ilarimikkonen
ilarimikkonen approved these changes Apr 26, 2019
View reviewed changes
Copy link
Member

@ilarimikkonen ilarimikkonen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved

@ilarimikkonen ilarimikkonen merged commit cf7d317 into develop Apr 26, 2019
@ilarimikkonen ilarimikkonen deleted the hack/tenant-mock branch April 26, 2019 10:38
@ghost ghost removed the in progress label Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ilarimikkonen ilarimikkonen ilarimikkonen approved these changes

Assignees

@matleppa matleppa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@matleppa @ilarimikkonen