Skip to content

OpenAI integration #1

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

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

OpenAI integration #1

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0677686
Updated Hello World message
May 23, 2025
71fed8f
OpenAI chat integration
May 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Block/Example.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,6 @@ class Example extends Template
* @return string
*/
public function getContent() {
return "Hello World";
return "Hello World, Welcome to Magento";
}
}
11 changes: 3 additions & 8 deletions etc/frontend/routes.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,3 @@
<?xml version="1.0"?>
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:App/etc/routes.xsd">
<router id="standard">
<route id="example" frontName="example">
<module name="MageSheet_Example" />
</route>
</router>
</config>
<route id='openai' frontName='openai'>
Copy link
Owner Author

Choose a reason for hiding this comment

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

The routes.xml file structure is incorrect. It is missing the root tag and the tag. A tag must be nested within a tag, which in turn must be within a tag. Please refer to the Magento documentation for the correct structure.

<module name='MageSheet_Example' />
</route>
10 changes: 10 additions & 0 deletions view/frontend/templates/chat.phtml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
<form id='chat-form'>
<input type='text' id='chat-input' placeholder='Type your message here...'/>
<button type='submit'>Send</button>
</form>
<div id='chat-output'></div>
<script type='text/javascript'>
require(['MageSheet_Example/js/chat'], function(chat) {
chat({}, document.getElementById('chat-form'));
});
</script>
17 changes: 17 additions & 0 deletions view/frontend/web/js/chat.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
define(['jquery'], function($) {
'use strict';
return function(config, element) {
.on('submit', function(event) {
Copy link
Owner Author

Choose a reason for hiding this comment

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

Burada element değişkeni kullanılmamış ve jQuery seçicisi eksik. Örneğin $(element).on('submit', ...) olarak kullanılmalı. Ayrıca, ilerleyen satırlarda da benzer şekilde jQuery seçicileri ($(element).find(...), $('#chat-output') gibi) eksik veya hatalı. Kodun doğru çalışması için form ve input elemanlarına net şekilde erişilmeli.

Copy link
Owner Author

Choose a reason for hiding this comment

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

There's a syntax error here. The .on('submit', ...) method is called without a preceding jQuery object. It should likely be $(element).on('submit', ...) if element refers to the form, or $(element).find('#chat-form').on('submit', ...) if element is a parent container.

event.preventDefault();
var message = .val();
Copy link
Owner Author

Choose a reason for hiding this comment

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

The message variable assignment is incomplete. It should be var message = $(element).find('#chat-input').val(); or similar, depending on how element is used and the structure of your HTML.

$.ajax({
Copy link
Owner Author

Choose a reason for hiding this comment

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

Burada message = .val(); ifadesi eksik. Doğru şekilde input elemanını seçip değerini almak için var message = $(element).find('#chat-input').val(); şeklinde kullanılmalı.

url: '/openai/chat',
type: 'POST',
data: { message: message },
success: function(response) {
.append('<div>' + response + '</div>');
Copy link
Owner Author

Choose a reason for hiding this comment

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

Burada append('<div>' + response + '</div>') ifadesinden önce hedef DOM elementi seçilmemiş. Yanıtı ekleyeceğiniz elementi (örn. #chat-output) net bir şekilde seçmelisiniz: $('#chat-output').append(...) gibi.

}
Copy link
Owner Author

Choose a reason for hiding this comment

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

Similar to previous issues, the jQuery selector is missing here. It should be $(element).find('#chat-output').append(...) to target the correct element.

});
});
};
});