#10 Add the X-WR-CALNAME property to the ICS

已合併
rsd 6 年之前 將 1 次代碼提交從 mmu_man/calname合併至 rsd/master
mmu_man commented 6 年之前

Giggity doesn't like when it's missing.

[Giggity](https://github.com/Wilm0r/giggity) doesn't like when it's missing.
siddharth ravikumar commented 6 年之前
所有者

@mmu_man, Thanks for the PR!

The patch looks good except for two minor nits:

  • To maintain uniformity with the rest of the self.cal.add calls in __init__, it would be nice to make X-WR-CALNAME lowercase (x-wr-calname).
  • Instead of the lp_year, it would be nice to use self.lp_year.
@mmu_man, Thanks for the PR! The patch looks good except for two minor nits: * To maintain uniformity with the rest of the `self.cal.add` calls in `__init__`, it would be nice to make `X-WR-CALNAME` lowercase (`x-wr-calname`). * Instead of the `lp_year`, it would be nice to use `self.lp_year`.
mmu_man commented 6 年之前
發佈者

Oh right, the lib upcases the keys by itself.

Oh right, the lib upcases the keys by itself.
mmu_man commented 6 年之前
發佈者

There you go :)

There you go :)
rsd6 年之前 關閉
siddharth ravikumar commented 6 年之前
所有者

Thank you @mmu_man!

Thank you @mmu_man!
該合併請求已經成功合併!
Sign in to join this conversation.
未選擇標籤
bug
未選擇里程碑
未指派成員
2 參與者
正在加載...
取消
保存
尚未有任何內容