-
Notifications
You must be signed in to change notification settings - Fork 79
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
Use cache for datastructures of the user/permission management #89
Conversation
09ba90a
to
c04ad95
Compare
@zaidka Any opinion about this? |
@zaidka No interest in improving the GUI? |
Oliver,
Do not confuse lack of time with lack of interest :)
…-dan
On Wed, Nov 7, 2018 at 6:25 AM Oliver Kraitschy ***@***.***> wrote:
@zaidka <https://github.com/zaidka> No interest in improving the GUI?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#89 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAyrgwchbH4DAtmnoWj71VBsY7_nGO-5ks5usvtWgaJpZM4Q33nQ>
.
|
@akcoder Does it take so long to write a short reply? No reply in almost a year doesn't look like much interest. |
There could be any number of reasons why Ziad hasn't replied. The email
could be buried in his inbox, he could have other work or life commitments.
Hell, I've personally had emails I've been meaning to reply to for many,
many months sitting in my inbox.
…-dan
On Thu, Nov 8, 2018 at 12:38 PM Oliver Kraitschy ***@***.***> wrote:
@akcoder <https://github.com/akcoder> Does it take so long to write a
short reply? No reply in almost a year doesn't look like much interest.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#89 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAyrg9ffa0ZnOMmzh_CWIhVM7IQMrKUjks5utKRYgaJpZM4Q33nQ>
.
|
@zaidka Wow, that was an answer 😀 Thank you! I have another improvement for this "feature" in the pipeline. Will create a PR tomorrow. |
I'm sorry :( Been very busy and overwhelmed with development and support. I keep postponing things until they get buried and I forget about them. I have merged this now. There's a few interesting things that I have been working on in hiding for the past year. Of which is a complete rewrite of the UI and a new API and query language to go with that. Will be sharing more details soon-ish. Sorry again for not being very responsive. And thanks for the PR. |
No problem! Slight but important improvement submitted in #102. |
This PR utilizes the rails cache to store the datastructures of the user/permission management of GenieACS GUI. The cache is cleared if there was a change related to users, roles or permissions.