Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

get_keys() must be removed or reworked #75

Open
quentinguidee opened this issue Nov 19, 2019 · 2 comments
Open

get_keys() must be removed or reworked #75

quentinguidee opened this issue Nov 19, 2019 · 2 comments
Labels
Type: Change

Comments

@quentinguidee
Copy link
Member

quentinguidee commented Nov 19, 2019 •
edited
Loading

Describe the bug

Because of this : numworks#1201, #32 must be partially removed.

Environment

  • Omega Version: 1.17
@quentinguidee quentinguidee added the Type: Bug Something isn't working label Nov 19, 2019
@quentinguidee quentinguidee removed this from To-Do in Omega 1.17 (released) Dec 5, 2019
@quentinguidee quentinguidee changed the title get_keys() - fix needed get_keys() must be removed Dec 5, 2019
@quentinguidee quentinguidee added this to To do in Omega 1.18 (released) via automation Dec 5, 2019
@quentinguidee quentinguidee changed the title get_keys() must be removed get_keys() must be removed or reworked Dec 5, 2019
@RedGl0w
Copy link
Member

RedGl0w commented Dec 22, 2019

Let's wait until the new version is released, and then we will have to delete it

@quentinguidee quentinguidee removed this from To do in Omega 1.18 (released) Dec 24, 2019
@quentinguidee quentinguidee added Type: Change and removed Status: On hold Type: Bug Something isn't working labels Dec 29, 2019
@RedGl0w
Copy link
Member

RedGl0w commented Jan 1, 2020

I think we shouldn't remove it until the new Ion module is released by epsilon, because actually, the kandinsky get_keys is more universal than the new Ion get_key...

@quentinguidee quentinguidee removed this from To do in Omega 1.19 (released) Feb 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Type: Change
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@quentinguidee @RedGl0w