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

Updated resource ACL #63

Merged
merged 1 commit into from
Apr 14, 2017
Merged

Updated resource ACL #63

merged 1 commit into from
Apr 14, 2017

Conversation

chedaroo
Copy link
Contributor

After installation exception being thrown when trying to access store configuration:

Invalid XML in file {...}/bitexpert/magento2-force-customer-login/etc/adminhtml/system.xml:
Element 'resource': [facet 'pattern'] The value 'bitExpert::ForceCustomerLogin' is not accepted by the pattern '([A-Z]+[a-zA-Z0-9]{1,}){1,}_[A-Z]+[A-Z0-9a-z]{1,}::[A-Za-z_0-9]{1,}'. Line: 11
Element 'resource': 'bitExpert::ForceCustomerLogin' is not a valid value of the atomic type 'typeAclResourceId'. Line: 11

Appears to be due to incorrect value for resource

After installation exception being thrown when trying to access store configuration:

```ssh
Invalid XML in file {...}/bitexpert/magento2-force-customer-login/etc/adminhtml/system.xml:
Element 'resource': [facet 'pattern'] The value 'bitExpert::ForceCustomerLogin' is not accepted by the pattern '([A-Z]+[a-zA-Z0-9]{1,}){1,}_[A-Z]+[A-Z0-9a-z]{1,}::[A-Za-z_0-9]{1,}'. Line: 11
Element 'resource': 'bitExpert::ForceCustomerLogin' is not a valid value of the atomic type 'typeAclResourceId'. Line: 11
```

Appears to be due to incorrect value for resource
@der-workfloh der-workfloh merged commit 87418ac into bitExpert:master Apr 14, 2017
@der-workfloh
Copy link
Contributor

Hi @chedaroo,

Thank you for your contribution! Your changes have been merged.

@der-workfloh der-workfloh added this to the 2.0 milestone Apr 14, 2017
@der-workfloh der-workfloh self-assigned this Apr 14, 2017
@adamj88
Copy link
Contributor

adamj88 commented Apr 14, 2017

@websharp thanks for merging, could you release this update as 2.0.0-RC3 please?

If you have a roadmap for 2.0.0 and beyond I may be able to help out.

@der-workfloh
Copy link
Contributor

Hi @adamj88 ,

2.0.0-RC3 has been released today, happy to get feedback if everything works now as expected.

For the moment there is no clear roadmap, but I would gladly discuss feature requests (by issue or PR) if you think the module would benefit from that. If you want to discuss in private, you could address this to @shochdoerfer or myself.

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.

3 participants