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

Module Creator Owner/Name + Notification Help Text Includes Warning Not To Use The Word "Oqtane". Fixes #3597 #3598

Merged
merged 3 commits into from
Jan 19, 2024

Conversation

thabaum
Copy link
Contributor

@thabaum thabaum commented Dec 29, 2023

Fixes #3597

This PR adds the information to the help text and notification to not use module owner or names that contain the word "oqtane".

It should help resolve issues as shown in #3597 while trying to create a module with a module name that contains "Oqtane" word in it.

This would need to be edited to inform the module name and owner is not allowed to start with the word "Oqtane" if current modifications being tested allow this without experiencing namespace conflicts in the framework itself. This is being discussed in #3597

Additional Context Relating To Documentation:
Another option to help enhance on-boarding module creators in this area is to have a link to Oqtane Module Creator Documentation for additional online support. I have an enhanced documentation outline proposed that would have a place to put a page or section dedicated to module development. This could also explain what you can/can't do such as using the word "oqtane" for module owner and name along with the best practices mentioned in #3597 for Oqtane Module Developers.

I can edit this PR for any updates to be merged relating or you may close this and rewrite things as needed if works best.

Cheers!

@thabaum thabaum changed the title Module Creator Owner/Name + Notification Help Text Includes Warning Not To Use The Word "Oqtane". Fixes # Module Creator Owner/Name + Notification Help Text Includes Warning Not To Use The Word "Oqtane". Fixes #3597 Dec 29, 2023
@thabaum thabaum mentioned this pull request Jan 6, 2024
@sbwalker sbwalker merged commit 584a7eb into oqtane:dev Jan 19, 2024
1 check passed
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.

Module Management bug
2 participants