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.

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

Jump to bottom

🧪 exp: gql types #262

Draft
wants to merge 27 commits into
base: develop
Choose a base branch
from
Draft

🧪 exp: gql types #262

wants to merge 27 commits into from

Conversation

melMass
Copy link
Member

@melMass melMass commented Feb 21, 2023 •
edited
Loading

Warning
Do not merge. This is a W.I.P

  • convert to typescript (wip)
  • add API type generation on build (let's keep in in version control src/bindings.ts, thanks to husky it should only update when teztok changes something)
  • add eslint on push (we should maybe scope it to some branches)

update from may 24

  • finish or revert (loosen) the typescript conversion. All recent commits where done using --no-verify
  • related to the first task, but I think the whole codegen thing was a bad idea after playing with it in teztok/teia's context. so same either finish or remove.
  • clean leftovers from the UI update.

@melMass melMass marked this pull request as draft February 22, 2023 01:26
Base automatically changed from develop to main February 22, 2023 14:56
@melMass melMass changed the base branch from main to develop February 22, 2023 15:35
@melMass melMass force-pushed the dev/mel-gql-experiment branch from 084b83a to 219b1df Compare February 22, 2023 16:06
@melMass melMass force-pushed the dev/mel-gql-experiment branch from 24b38eb to 8946826 Compare February 22, 2023 23:28
@melMass melMass mentioned this pull request Mar 4, 2023
Draft
4 tasks
@melMass melMass added 🛠 developer Changes that affect developers only, including non-customer-facing tools (e.g. changes to bin/setup) 🎯 targeting develop Temp label to identify PRs meant to be merged in develop labels Mar 4, 2023
melMass added 3 commits March 13, 2023 01:51
@melMass
fix: ⚡️ from merge
0ae0639
@melMass
Merge branch 'develop' into dev/mel-gql-experiment
bc588e9
@melMass
fix: 🐛 build
d6f65d5 
the config probably need a bit of cleaning at some point.
There is a misconfiguration between ts, vite and eslint
@melMass
Copy link
Member Author

melMass commented May 4, 2023

Alright, I'll finish this this week!

@melMass melMass force-pushed the dev/mel-gql-experiment branch from 841a2e4 to d6f65d5 Compare May 4, 2023 16:47
@melMass
Copy link
Member Author

melMass commented May 24, 2023

Alright, I'll finish this this week!

@melMass melMass mentioned this pull request Jul 26, 2023
Closed
@github-actions
Copy link

github-actions bot commented Aug 9, 2023

PR Preview Action v1.4.4
🚀 Deployed preview to https://teia-community.github.io/teia-ui/previews/pr-262/
on branch gh-pages at 2023-08-09 22:40 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🛠 developer Changes that affect developers only, including non-customer-facing tools (e.g. changes to bin/setup) 🎯 targeting develop Temp label to identify PRs meant to be merged in develop
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@melMass