Code Monkey home page Code Monkey logo

Comments (11)

ChihSeanHsu avatar ChihSeanHsu commented on August 19, 2024 1

這issue有人接嗎?沒有的話可以幫忙。

from go-bbs.

ChihSeanHsu avatar ChihSeanHsu commented on August 19, 2024 1

Hi @PichuChen,
我覺得有可能是function命名的問題,因為目前是照著原本的bbs.c做些變動搬過來的,我一直覺得名字有點不太合適。

因為從bbs.c裡面看起來是轉發文章跟推文都是在最後呼叫這個do_add_recommend
所以從那個角度來看,轉發文章的系統資訊也算是一種推文。

但如果說要更加一般化的話,我認為是可以把那些recommend的數目跟還有已讀重算這些拔出來,等到實作推文的時候再去實作他們,這個function單純就是處理增加一行文字到檔案的最底部就好。

也就是只傳入direct, fileHeader, buf這三個東西。

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

目前這個可能要同時等 cross_post do_add_recomment 的函式註解完成 https://docs.google.com/spreadsheets/d/1ITDYuj6TjeodZDtnXPrctZ6We6Mz_tXosmR_O1TCkXo/edit#gid=0

這部分可能要聯絡 https://github.com/wagaru dogeTW 協調喔

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

誒不對,好像是 do_add_recomment ,這個 doge 那邊還沒有回應,所以你如果要做的話可以先在 Google Sheet 上面提示一下你那邊會接手

from go-bbs.

ChihSeanHsu avatar ChihSeanHsu commented on August 19, 2024

好的,有兩個問題想要問一下,

  1. 這個method(暫稱AppendNewLine)會讓do_add_recommentcross_post共用?
  2. do_add_recomment 有開相關的issue或PR嗎?

感謝

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

好的,有兩個問題想要問一下,

  1. 這個method(暫稱AppendNewLine)會讓do_add_recommentcross_post共用?
  2. do_add_recomment 有開相關的issue或PR嗎?

感謝

這個method不會使用到 do_add_recomment 或是被 do_add_recomment 使用,他會像是實作 Golang 版本的 do_add_Recomment (原版是C語言的版本)

from go-bbs.

ChihSeanHsu avatar ChihSeanHsu commented on August 19, 2024

好的,有兩個問題想要問一下,

  1. 這個method(暫稱AppendNewLine)會讓do_add_recommentcross_post共用?
  2. do_add_recomment 有開相關的issue或PR嗎?

感謝

這個method不會使用到 do_add_recomment 或是被 do_add_recomment 使用,他會像是實作 Golang 版本的 do_add_Recomment (原版是C語言的版本)

了解了,剛剛稍微去看了一下bbs.c,
感謝你的回應。

from go-bbs.

ChihSeanHsu avatar ChihSeanHsu commented on August 19, 2024

Hi @PichuChen ,

目前已經在google sheet裡填入do_add_recommend的註解了。

準備開始實作,但可能這個issue會跟#83有些相依,需要使用到那邊的connector,
或是這個會是個獨立的connector?

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

@ChihSeanHsu 他應該可以是個獨立的 Connector, 或是兩邊誰後近來在處理merge即可

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

@ChihSeanHsu 我發現你那邊目前實作的函式不夠一般化,他有點太針對推文了,我原先的想法是如果可以新增一個函式,他是可以自定義內容的,那這樣轉發文章的系統訊息就可以用這個函式來完成了。

而接著在使用者推文的部分則是呼叫這個函式進行就行了。
不知道你覺得這個想法如何?

from go-bbs.

PichuChen avatar PichuChen commented on August 19, 2024

@ChihSeanHsu 我覺得把算數目(或是說設定數目)和文章內容修改切開很OK

from go-bbs.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.