Hu-Fi / Mr.Market

Mr. Market is the exchange oracle of HuFi, and a CeFi crypto bot on Mixin Messenger
https://mr-market-one.vercel.app
GNU Affero General Public License v3.0
1 stars 6 forks source link

update deployment link #182

Closed zed-wong closed 4 months ago

zed-wong commented 4 months ago

User description

Description

Updated deployment link

Summary of changes

Fixed vercel deployment

How to test the changes

Try it

Related issues

Fix deploy #179


PR Type

documentation, enhancement


Description


Changes walkthrough ๐Ÿ“

Relevant files
Documentation
README.md
Update Vercel deployment link with build and install commands

README.md
  • Updated the Vercel deployment link to include build and install
    commands.
  • +1/-1     

    ๐Ÿ’ก PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

    vercel[bot] commented 4 months ago

    The latest updates on your projects. Learn more about Vercel for Git โ†—๏ธŽ

    Name Status Preview Comments Updated (UTC)
    mr-market โœ… Ready (Inspect) Visit Preview ๐Ÿ’ฌ Add feedback May 23, 2024 6:42am
    railway-app[bot] commented 4 months ago

    ๐Ÿš… Previously deployed to Railway in the quiet-desk project. Environment has been deleted.

    github-actions[bot] commented 4 months ago

    PR Description updated to latest commit (https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7)

    github-actions[bot] commented 4 months ago

    PR Description updated to latest commit (https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7)

    github-actions[bot] commented 4 months ago

    PR Review ๐Ÿ”

    (Review updated until commit https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7)

    โฑ๏ธ Estimated effort to review [1-5] 1, because the PR involves a simple change in the deployment link in the README.md file. The change is straightforward and does not involve complex code modifications.
    ๐Ÿงช Relevant tests No
    โšก Possible issues No
    ๐Ÿ”’ Security concerns No
    Code feedback:
    github-actions[bot] commented 4 months ago

    Persistent review updated to latest commit https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7

    github-actions[bot] commented 4 months ago

    PR Code Suggestions โœจ

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Adjust the width of the Vercel deployment button to match the Railway button for visual consistency ___ **Consider using consistent image sizes for deployment buttons to maintain a uniform look in
    the README. The Vercel button is currently set to 72px width, which might not align well
    visually with the Railway button set at 96px width.** [README.md [9]](https://github.com/Hu-Fi/Mr.Market/pull/182/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R9-R9) ```diff -Deploy with Vercel +Deploy with Vercel ```
    Suggestion importance[1-10]: 7 Why: The suggestion improves the visual consistency of the README by ensuring that the deployment buttons have the same width, which enhances readability and aesthetics. However, this is a minor enhancement and does not address any functional issues.
    7
    github-actions[bot] commented 4 months ago

    PR Code Suggestions โœจ

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Adjust the width of the Vercel deployment button to match the Railway button for visual consistency ___ **Consider using consistent image sizes for deployment buttons to maintain a uniform
    appearance in the README. The Vercel deployment button has a width of "72px", which is
    smaller compared to the Railway button's "96px". Adjusting the Vercel button to match the
    Railway button's width can improve visual consistency.** [README.md [9]](https://github.com/Hu-Fi/Mr.Market/pull/182/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R9-R9) ```diff -Deploy with Vercel +Deploy with Vercel ```
    Suggestion importance[1-10]: 7 Why: The suggestion improves the visual consistency of the README by making the deployment buttons uniform in size. This is a minor enhancement but contributes to better readability and aesthetics.
    7
    github-actions[bot] commented 4 months ago

    PR Description updated to latest commit (https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7)

    github-actions[bot] commented 4 months ago

    Persistent review updated to latest commit https://github.com/Hu-Fi/Mr.Market/commit/eaa6ca02b20487f152eaf112088690a7108b78e7

    github-actions[bot] commented 4 months ago

    PR Code Suggestions โœจ

    CategorySuggestion                                                                                                                                    Score
    Enhancement
    Adjust the width of the Vercel deployment button to match the Railway button for visual consistency ___ **Consider using consistent image sizes for deployment buttons to maintain a uniform
    appearance. The Vercel deployment button has a width of 72px, while the Railway deployment
    button is 96px wide. Adjusting both to the same width can improve the visual consistency
    of the document.** [README.md [9-10]](https://github.com/Hu-Fi/Mr.Market/pull/182/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R9-R10) ```diff -Deploy with Vercel +Deploy with Vercel Deploy on Railway ```
    Suggestion importance[1-10]: 7 Why: The suggestion improves the visual consistency of the document, which is a minor enhancement. It does not address any major bugs or security concerns but contributes to better readability and aesthetics.
    7