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

Oracle tools #161

Merged
merged 22 commits into from
Apr 3, 2024
Merged

Oracle tools #161

merged 22 commits into from
Apr 3, 2024

Conversation

Mond1c
Copy link
Contributor

@Mond1c Mond1c commented Mar 20, 2024

I will add Calibrator as a frontend compoent.

Locator is now separted from the admin frontend. And I want to know, is it a good idea? Or I need to get it back?

@Mond1c Mond1c marked this pull request as draft March 20, 2024 10:29
@Mond1c Mond1c changed the title Sniper locator controller Sniper locator controller (refactored version) Mar 22, 2024
@Mond1c Mond1c marked this pull request as ready for review March 26, 2024 09:21
@Mond1c Mond1c force-pushed the new-sniper-locator branch from 0b05658 to adc2af2 Compare April 1, 2024 05:21
@Mond1c Mond1c force-pushed the new-sniper-locator branch from dc371da to 9464164 Compare April 1, 2024 05:39
@Mond1c
Copy link
Contributor Author

Mond1c commented Apr 1, 2024

I've created package.json with exports in common folder. I will add others files to this folder, but now I want to know if can I solve the copypaste problem this way?

@Mond1c Mond1c force-pushed the new-sniper-locator branch from e95b698 to 85298c9 Compare April 3, 2024 09:31
@Mond1c Mond1c changed the title Sniper locator controller (refactored version) Oracle tools Apr 3, 2024
Comment on lines +1 to +9
import { AbstractWidgetService } from "shared-code/abstractWidget";
import { BASE_URL_BACKEND, ADMIN_ACTIONS_WS_URL } from "../config";

export class AbstractWidgetImpl extends AbstractWidgetService {
constructor(apiPath, errorHandler, listenWS) {
super(BASE_URL_BACKEND, ADMIN_ACTIONS_WS_URL, apiPath, errorHandler, listenWS);

}
}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now admin only has the implementation and I hope I didn't break anything

@Mond1c Mond1c force-pushed the new-sniper-locator branch from a7449f1 to 80fd1e0 Compare April 3, 2024 13:36
Copy link
Contributor

@kbats183 kbats183 left a comment

Choose a reason for hiding this comment

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

Good for begin
Sometimes we share more code from admin to common, but not today

@kbats183 kbats183 merged commit c5ff3a3 into icpc:main Apr 3, 2024
3 checks passed
MegaVerkruzo pushed a commit to MegaVerkruzo/live-v3 that referenced this pull request Apr 9, 2024
* init sniper locator controller

* fix

* fix type

* delete dependency

* change name

* fix bugs

* fix auth problem

* fix eslint

* added default username and password

* mini refactored

* fix mustExist flag + deleted useless get requests

* fixed linter

* create shared code

* fix locator in overlay

* rename to oracle-tools

* fix package name

* move abstractWidget.js to common folder

* make abstractWidgetImpl for admin

* added LOCATOR_MAGICAL_CONSTANT

* fixed bugs + move BasicAuthKey to object

* rename all parts of code from sniper to oracle

---------

Co-authored-by: Mihail Kornilovich <[email protected]>
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.

2 participants