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

New PR template #72

Closed
Keyrxng opened this issue Oct 28, 2024 · 11 comments · Fixed by #77
Closed

New PR template #72

Keyrxng opened this issue Oct 28, 2024 · 11 comments · Fixed by #77

Comments

@Keyrxng
Copy link
Member

Keyrxng commented Oct 28, 2024

Suggestion below. If I place the comment at the top I lose formatting for the three sections but if it can be solved it would be better at the top maybe.


Resolves https.../yourTaskURL


Changes:

  • ...

QA:

  • ...

<!--- 

Insert the number of the task you are completing if within this repository or paste the url of the task

Resolves https:... OR #<taskNumber>

/**
We advise that you give a brief description of your changes either that it meets
the specification exactly as described or what you had to do outwith the spec to resolve it. 
This is not required but again, it is advised.
*/

Changes:
- I had to do abc because of xyz but the spec was implemented otherwise

/**
NOTICE: This is required for all pull requests and will be requested by reviewers if not present.

- Include a screenshot/video or some other visual confirmation that your changes solve the task
- If this is not applicable, build a unit test to prove it works as intended
*/
QA:
<url to your plugin fork or test issue in your own org showcasing the end result>
<screenshot of some kind of UI fix>
<E2E video of using the software>
Copy link

ubiquity-os bot commented Oct 28, 2024

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

Copy link

ubiquity-os bot commented Oct 28, 2024

- error: missing required argument 'address'

@imabutahersiddik
Copy link

/wallet 0x7245f5cb278ea948ab6302bd911db00ad4889672

Copy link

ubiquity-os bot commented Oct 28, 2024

+ Successfully registered wallet address

@imabutahersiddik
Copy link

/start

Copy link

ubiquity-os bot commented Oct 28, 2024

Deadline Mon, Oct 28, 5:35 PM UTC
Beneficiary 0x7245F5Cb278ea948Ab6302Bd911db00Ad4889672

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

/help

Copy link

ubiquity-os bot commented Nov 4, 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

Copy link

ubiquity-os bot commented Nov 4, 2024

Passed the deadline and no activity is detected, removing assignees: @imabutahersiddik.

Copy link

ubiquity-os bot commented Nov 8, 2024

Deadline Fri, Nov 8, 2:25 PM UTC
Beneficiary 0xbB689fDAbBfc0ae9102863E011D3f897b079c80F

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.

Copy link

ubiquity-os bot commented Nov 10, 2024

 [ 6 WXDAI ] 

@zugdev
Contributions Overview
ViewContributionCountReward
IssueTask16
ReviewComment20
Conversation Incentives
CommentFormattingRelevanceReward
Resolves #72 Changes:- Refactored the pull template as speci…
6.5
content:
  content:
    p:
      score: 0
      elementCount: 7
    ul:
      score: 0
      elementCount: 1
    li:
      score: 0.5
      elementCount: 3
    a:
      score: 5
      elementCount: 1
  result: 6.5
regex:
  wordCount: 25
  wordValue: 0
  result: 0
0.90
If you press to edit my message above you can see the template's…
0
content:
  content:
    p:
      score: 0
      elementCount: 1
  result: 0
regex:
  wordCount: 34
  wordValue: 0
  result: 0
0.80

 [ 9.3 WXDAI ] 

@Keyrxng
Contributions Overview
ViewContributionCountReward
IssueSpecification19.3
Conversation Incentives
CommentFormattingRelevanceReward
Suggestion below. If I place the comment at the top I lose forma…
3.1
content:
  content:
    p:
      score: 0
      elementCount: 4
    hr:
      score: 0
      elementCount: 4
    ul:
      score: 0
      elementCount: 2
    li:
      score: 0.5
      elementCount: 2
  result: 1
regex:
  wordCount: 36
  wordValue: 0.1
  result: 2.1
19.3

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