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

Auth check #268

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

jgeissler-letsdev
Copy link

Pull Request Template

Description of Changes

Please provide a brief summary of what you have changed in this pull request. Be clear and concise.

Related Issue

If this pull request is related to an existing issue, please link that issue using the format #issue_number.

BaSyx Configuration for Testing

Describe any specific configuration settings or environment setup required to test these changes effectively. If possible, please provide the BaSyx configuration as .zip file.

AAS Files Used for Testing

Please provide any AAS files that were used in the testing of these changes. Include a brief descriptions of their relevance to the changes being proposed.

Additional Information

Include any additional information or context that could be useful for the reviewer. This could include challenges faced, alternative solutions that were considered, etc.


Please ensure that you have tested your changes thoroughly before submitting the pull request.

@aaronzi
Copy link
Member

aaronzi commented Dec 7, 2024

Thank you for your PR. Could you please raise it in the correct GitHub repository? The BaSyx AAS Web UI was moved to its own repo a few months ago. You can find it here.

In addition to that, please sign the Eclipse Contributor Agreement (ECA). Please create an Eclipse Account for this.

It would be nice also to include a short description of what your pull request adds. Is it a feature, a fix, or something else?

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.

5 participants