Skip to content

Refactoring, translate to Russian, add CORS middleware#4

Merged
Fenny merged 17 commits intogofiber:masterfrom
koddr:master
Feb 3, 2020
Merged

Refactoring, translate to Russian, add CORS middleware#4
Fenny merged 17 commits intogofiber:masterfrom
koddr:master

Conversation

@koddr
Copy link
Copy Markdown

@koddr koddr commented Feb 1, 2020

  • Translate README to Russian lang
  • Clean logo.jpg (remove top gray line)
  • Create PNG logo with transparent background (two sizes: 320px, 750px)
  • Add dummy CORS middleware
  • Refactoring spaces and typo in comments

Also, I change names of some functions/structs (for go-lint recommendations):

  • route struct -> Route struct
  • BaseUrl -> BaseURL
  • OriginalUrl -> OriginalURL
  • Ip -> IP
  • Json -> JSON
  • Jsonp -> JSONP
  • Xml -> XML
  • JsonString -> JSONString
  • JsonBytes -> JSONBytes
  • contentTypeJson -> contentTypeJSON
  • contentTypeXml -> contentTypeXML

This is my support awesome project Fiber! 😀

P.S. If you can (and want), please add me to gofiber organization. I want to styling it (create and upload cute logo, description, pins, etc.), but I can't send PR to organization 😞

@koddr koddr requested a review from Fenny February 2, 2020 09:09
@Fenny
Copy link
Copy Markdown
Member

Fenny commented Feb 2, 2020

Thnx, I will review it later tonight 👍

Copy link
Copy Markdown
Member

@Fenny Fenny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pull 👍 , I will add a language structure to the API documentation later on.

@Fenny Fenny merged commit b7e63c4 into gofiber:master Feb 3, 2020
Fenny pushed a commit that referenced this pull request Feb 8, 2020
Accumulative update @ 08 Feb 2020
Fenny pushed a commit that referenced this pull request Feb 17, 2020
Accumulative update @ 16 Feb 2020
Fenny pushed a commit that referenced this pull request Mar 15, 2020
Accomodation update @ 15 Mar 2020
Fenny pushed a commit that referenced this pull request Oct 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants