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

Create add/remove config logic #3

Open
Tracked by #1
0x4007 opened this issue Sep 15, 2024 · 16 comments · May be fixed by #13
Open
Tracked by #1

Create add/remove config logic #3

0x4007 opened this issue Sep 15, 2024 · 16 comments · May be fixed by #13

Comments

@0x4007
Copy link
Member

0x4007 commented Sep 15, 2024

  1. We need to be able to select which ubiquibot-config.yml to edit, so we at minimum need an organization picker after GitHub authentication.
  2. We need to parse the YML file, and target the config based on the plugin's URL/location. Then edit the YML file and push the commit using the user's credentials.

More thoughts on the readme.

LLM

Without research, the lazy answer is that LLMs can easily solve the problem. We could even consider passing in the most important/relevant files of the plugin so that the LLM can figure out all the default values and other miscellaneous details.

But it would certainly be preferred to implement this with traditional code. I think with a bit of research it can be pretty straightforward.

@Neuman-Nz
Copy link

/help

Copy link

Available Commands

Command Description Example
/help List all available commands. /help
/allow Allows the user to modify the given label. /allow @user1 label
/query Returns the user's wallet, access, and multiplier information. /query @ubiquibot
/start Assign yourself to the issue. /start
/stop Unassign yourself from the issue. /stop
/wallet Register your wallet address for payments. /wallet ubq.eth

1 similar comment
Copy link

ubiquity-os bot commented Sep 27, 2024

Available Commands

Command Description Example
/help List all available commands. /help
/allow Allows the user to modify the given label. /allow @user1 label
/query Returns the user's wallet, access, and multiplier information. /query @ubiquibot
/start Assign yourself to the issue. /start
/stop Unassign yourself from the issue. /stop
/wallet Register your wallet address for payments. /wallet ubq.eth

@Neuman-Nz
Copy link

/start

Copy link

ubiquity-os bot commented Sep 27, 2024

! Please set your wallet address with the /wallet command first and try again.

Copy link

ubiquity-os bot commented Sep 27, 2024

! No wallet address found

@0x4007
Copy link
Member Author

0x4007 commented Sep 27, 2024

@gentlementlegen can you fix the double post?

@gentlementlegen
Copy link
Member

@Apetree100122

This comment was marked as spam.

Copy link

Deadline Tue, Oct 8, 5:33 AM UTC
Beneficiary 0xC6DcC5424AE41dec50e9444742f83Cd489D5527a

Tip

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

1 similar comment
Copy link

ubiquity-os bot commented Oct 7, 2024

Deadline Tue, Oct 8, 5:33 AM UTC
Beneficiary 0xC6DcC5424AE41dec50e9444742f83Cd489D5527a

Tip

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

Keyrxng pushed a commit to ubq-testing/ubiquity-os-plugin-installer that referenced this issue Nov 7, 2024
@Keyrxng
Copy link
Member

Keyrxng commented Nov 7, 2024

starting this now and I'll just link the PR again, it'll be building on top of #12.

@Keyrxng Keyrxng linked a pull request Nov 9, 2024 that will close this issue
Copy link

Deadline Sun, Nov 10, 6:01 AM UTC
Beneficiary 0xAe5D1F192013db889b1e2115A370aB133f359765

Tip

  • Use /wallet 0x0000...0000 if you want to update your registered payment wallet address.
  • Be sure to open a draft pull request as soon as possible to communicate updates on your progress.
  • Be sure to provide timely updates to us when requested, or you will be automatically unassigned from the task.

@gentlementlegen
Copy link
Member

gentlementlegen commented Nov 9, 2024

Funny, I had made a tool for my personal usage a while ago, I didn't remember this task.

@Keyrxng
Copy link
Member

Keyrxng commented Nov 9, 2024

Funny, I had made a tool for my personal usage a while ago, I didn't remember this task.

If it's better than my approach pls share and I'll use it here

@gentlementlegen
Copy link
Member

@Keyrxng It is a very different approach! At first it was just for editing the configuration files in situ. Also I used NextJs and Material which I don't think is welcome haha.

@rndquu rndquu mentioned this issue Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants