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

Base #15

Closed
wants to merge 21 commits into from
Closed

Base #15

wants to merge 21 commits into from

Conversation

Xiong-stratos
Copy link

Description

Checklist

  • Ran Linting
  • Targeted PR against correct branch.
  • Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
  • Wrote unit tests.
  • Added an entry to the CHANGELOG.md file.
  • Re-reviewed Files changed in the Github PR explorer.

huichiaotsou and others added 21 commits July 14, 2022 10:00
@huichiaotsou
replace redundant delegatorAddresses[i] with variable delegatorAddress
b8fe20f
@huichiaotsou
add back missing dependency for useEffect in useProfilesRecoil
86a6d40
@huichiaotsou
add change log
6f40f34
@huichiaotsou
add issue link to change log
ce9f959
@yayay927
Merge pull request #903 from forbole/aaron/desmos_profile
6602059 
fix: desmos profile not showing on proposal details page
@yayay927
update change url files
01b698d
@yayay927
update changelog
5df6396
@yayay927
update changelog dates and new version
5158cd6
@yayay927
udpate general config new version
1bd59f0
@yayay927
add matomo url and id to docker production yml
c6394d3
@yayay927
update changelog
c0410f8
@yayay927
Merge pull request #972 from forbole/ellie/base-change-url
f79d7ba 
Base: add change url files + new version
@yayay927
fix bug
0a897ef
@yayay927
Merge pull request #1000 from forbole/ellie/release-base-v2.1.2
9457d60 
Base: fix bug from v2.1.1 and release v2.1.2
@saiSunkari19
fix: websocket url in .env.sample (#996)
e45102b 
<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v                               ✰  Thanks for creating a PR! ✰
v    Before smashing the submit button please review the checkboxes.
v If a checkbox is n/a - please still include it but + a little note why
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

## Description
This is a small fix, that is related to WebSocket URL in `.env.sample`


## Checklist
- [x] Ran Linting
- [x] Targeted PR against correct branch.
- [x] Linked to Github issue with discussion and accepted design OR link
to spec that describes this work.
- [ ] Wrote unit tests.
- [ ] Added an entry to the `CHANGELOG.md` file.
- [ ] Re-reviewed `Files changed` in the Github PR explorer.
@RiccardoM
build(ci): added Mergify config (#1016)
119eaba 
<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v                               ✰  Thanks for creating a PR! ✰
v    Before smashing the submit button please review the checkboxes.
v If a checkbox is n/a - please still include it but + a little note why
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

## Description

This PR adds the configuration to properly support
[Mergify](https://mergify.com/) in order to be able to automatically
merge PR requests once the following checks have passed:

- the base branch in set to either `base` or `chains/*` 
- all the PR checks within GitHub are passing 
- the PR has the `automerge` label


## Checklist
- [ ] Ran Linting
- [ ] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link
to spec that describes this work.
- [ ] Wrote unit tests.
- [ ] Added an entry to the `CHANGELOG.md` file.
- [ ] Re-reviewed `Files changed` in the Github PR explorer.
@RiccardoM
build(ci): improve PR process (#1015)
9125601 
<!-- < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < < ☺
v                               ✰  Thanks for creating a PR! ✰
v    Before smashing the submit button please review the checkboxes.
v If a checkbox is n/a - please still include it but + a little note why
☺ > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >  -->

## Description

This PR improves the overall PR creation and checking process.
Particularly it includes the following changes:
- improved the `PULL_REQUEST_TEMPLATE` to have more meaningful
checkboxes that the author needs to check
- added a GitHub workflow to make sure that PR titles follow the
[Conventional Commits](https://www.conventionalcommits.org/en/v1.0.0/)
specification


## Checklist
- [ ] Ran Linting
- [ ] Targeted PR against correct branch.
- [ ] Linked to Github issue with discussion and accepted design OR link
to spec that describes this work.
- [ ] Wrote unit tests.
- [ ] Added an entry to the `CHANGELOG.md` file.
- [ ] Re-reviewed `Files changed` in the Github PR explorer.
@teamchong
feat: add Sentry.io issue tracking (#1006)
278bcc3 
https://docs.sentry.io/platforms/javascript/guides/nextjs/

Closes #1005 

## Checklist
- [X] Ran Linting
- [X] Targeted PR against correct branch.
- [X] Linked to Github issue with discussion and accepted design OR link
to spec that describes this work.
- [ ] Wrote unit tests.
- [X] Added an entry to the `CHANGELOG.md` file.
- [X] Re-reviewed `Files changed` in the Github PR explorer.
@MonikaCat
build: switch from NPM to Yarn (#1028)
e1f8d0e 
## Description
This PR aims to migrate from npm to yarn berry package manager.

Closes: #XXXX
[BDU-631](https://forbole.atlassian.net/browse/BDU-631) 
<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

### Author Checklist

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [ ] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
@yayay927
feat: add transaction type and messages count (#1034)
9decb57 
## Description

Closes: [BDU-35](https://forbole.atlassian.net/browse/BDU-35)

This pr is for adding message type into transactions page and latest transactions component in home page. 



---

### Author Checklist

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
@MonikaCat
feat: increment unit tests coverage (#1044)
a13f442 
## Description

Closes: #XXXX
[BDU-580](https://forbole.atlassian.net/browse/BDU-580)

<!-- Add a description of the changes that this PR introduces and the
files that
are the most critical to review. -->

---

### Author Checklist

*All items are required. Please add a note to the item if the item is
not applicable and
please add links to any relevant follow up issues.*

I have...

- [x] ran linting
- [x] wrote tests where necessary
- [x] included the correct [type
prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json)
in the PR title
- [x] targeted the correct branch
- [x] provided a link to the relevant issue or specification
- [x] reviewed "Files changed" and left comments if necessary
- [x] confirmed all CI checks have passed
- [x] added an entry to the `CHANGELOG.md` file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WangLiStratos WangLiStratos WangLiStratos approved these changes

@weichen-qsnetwork weichen-qsnetwork Awaiting requested review from weichen-qsnetwork

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@Xiong-stratos @WangLiStratos @huichiaotsou @yayay927 @saiSunkari19 @RiccardoM @teamchong @MonikaCat